Skip to content

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jsync-swirlds
Copy link
Contributor

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

@jsync-swirlds jsync-swirlds self-assigned this Mar 7, 2025
@jsync-swirlds jsync-swirlds requested a review from mgarbs as a code owner March 7, 2025 00:47
Copy link

netlify bot commented Mar 7, 2025

Deploy Preview for hedera-hips ready!

Name Link
🔨 Latest commit 395dd44
🔍 Latest deploy log https://app.netlify.com/sites/hedera-hips/deploys/67dc4ef318710300080a2226
😎 Deploy Preview https://deploy-preview-1137--hedera-hips.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@jsync-swirlds jsync-swirlds force-pushed the block-node-discoverability branch from dcf2b1f to 0f8a0b2 Compare March 7, 2025 00:51
@jsync-swirlds jsync-swirlds changed the title Block Node Discoverability HIP-1137: Block Node Discoverability Mar 7, 2025
@jsync-swirlds jsync-swirlds force-pushed the block-node-discoverability branch 2 times, most recently from a69413b to c03c8b3 Compare March 7, 2025 00:55
Copy link
Contributor

@Nana-EC Nana-EC left a 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

@jsync-swirlds jsync-swirlds force-pushed the block-node-discoverability branch 2 times, most recently from cfc1ba1 to 91c3ab0 Compare March 7, 2025 18:50
Copy link
Contributor

@Nana-EC Nana-EC left a 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.

@jsync-swirlds jsync-swirlds force-pushed the block-node-discoverability branch from 91c3ab0 to 126f9d7 Compare March 14, 2025 20:19
Copy link
Contributor

@Nana-EC Nana-EC left a 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

@jsync-swirlds jsync-swirlds force-pushed the block-node-discoverability branch 4 times, most recently from 7312b93 to 395dd44 Compare March 20, 2025 17:22
@jsync-swirlds jsync-swirlds force-pushed the block-node-discoverability branch from 395dd44 to 797d320 Compare March 26, 2025 17:59
@jsync-swirlds jsync-swirlds requested a review from a team as a code owner March 26, 2025 17:59
@jsync-swirlds jsync-swirlds requested a review from Nana-EC April 9, 2025 16:15
@jsync-swirlds jsync-swirlds force-pushed the block-node-discoverability branch 3 times, most recently from e109dc8 to 4d0495a Compare April 9, 2025 16:24
@jsync-swirlds jsync-swirlds force-pushed the block-node-discoverability branch from 4d0495a to 6b564f5 Compare April 25, 2025 18:12
* 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>
@jsync-swirlds jsync-swirlds force-pushed the block-node-discoverability branch from 6b564f5 to b92c0a6 Compare May 28, 2025 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants