You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
roman-khimov opened this issue
Mar 15, 2023
· 0 comments
Labels
configConfiguration format update or breaking changefeatureCompletely new functionalityI2Regular impactneofs-irInner Ring node application issuesS2Regular significanceU3Regular
Is your feature request related to a problem? Please describe.
Current node validation logic mixes logical correctness and policy checks. For example, address validation is just a correctness checks, same thing for maintenance mode. However locode check is a policy, networks may want to do it or may not want to. They at the same time may want to check some network-specific attributes according to any network-specific logic.
Describe the solution you'd like
Make embedded policy checks configurable and provide a generic extension hook to check node data via an external program.
Describe alternatives you've considered
Embedded ones can be moved out of the node as well, but maybe that'd make them a bit less convenient to use.
The text was updated successfully, but these errors were encountered:
configConfiguration format update or breaking changefeatureCompletely new functionalityI2Regular impactneofs-irInner Ring node application issuesS2Regular significanceU3Regular
Is your feature request related to a problem? Please describe.
Current node validation logic mixes logical correctness and policy checks. For example, address validation is just a correctness checks, same thing for maintenance mode. However locode check is a policy, networks may want to do it or may not want to. They at the same time may want to check some network-specific attributes according to any network-specific logic.
Describe the solution you'd like
Make embedded policy checks configurable and provide a generic extension hook to check node data via an external program.
Describe alternatives you've considered
Embedded ones can be moved out of the node as well, but maybe that'd make them a bit less convenient to use.
The text was updated successfully, but these errors were encountered: