-
Notifications
You must be signed in to change notification settings - Fork 152
HIP-1137: Block Node Discoverability #1137
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for hedera-hips ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
dcf2b1f
to
0f8a0b2
Compare
a69413b
to
c03c8b3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice push, add initial thoughts
cfc1ba1
to
91c3ab0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updates look good, look forward to additional details.
91c3ab0
to
126f9d7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice improvements. Added some new suggestions
7312b93
to
395dd44
Compare
395dd44
to
797d320
Compare
e109dc8
to
4d0495a
Compare
4d0495a
to
6b564f5
Compare
* Add new HIP document describing discoverable nodes * This may be block nodes, rpc relays, or any other node that should be discoverable by any network participant. Signed-off-by: Joseph Sinclair <121976561+jsync-swirlds@users.noreply.github.com>
* Added missing "discoverable" for node update transaction message name. * Added missing detail in the "Discoverable Node" definition. * Clarified that account ID is not relevant to discoverable nodes. * Added a specification for the value in state used to store the discoverable node information. * Added detail for admin key usage, network throttles, and related information to the security implications section. * Added an SDK Considerations section. * Added Mirror Node as a node type. * Cleaned up some specification wording. * Added some detail to the "reputation" open issue. * Added some initial use cases, community ideas and revision are encouraged. Signed-off-by: Joseph Sinclair <121976561+jsync-swirlds@users.noreply.github.com>
6b564f5
to
b92c0a6
Compare
Description
A new HIP for describing discoverable nodes
These may be block nodes, rpc relays, or any other node that should be discoverable by any network participant.
Discussion
#1132