Skip to content

Update seeds from hnsnodes#134

Open
Nathanwoodburn wants to merge 1 commit into
handshake-org:masterfrom
Nathanwoodburn:update/seeds
Open

Update seeds from hnsnodes#134
Nathanwoodburn wants to merge 1 commit into
handshake-org:masterfrom
Nathanwoodburn:update/seeds

Conversation

@Nathanwoodburn

Copy link
Copy Markdown
Member

This removes old inaccessible peers and adds peers discovered using hnsnodes

@Nathanwoodburn Nathanwoodburn self-assigned this Jun 5, 2026
@Nathanwoodburn Nathanwoodburn force-pushed the update/seeds branch 2 times, most recently from 1984781 to cc745a5 Compare June 5, 2026 00:57
@shadstoneofficial

Copy link
Copy Markdown

Tested PR #134 from Bangkok on macOS.

Reachability looks much better from my network:

  • current master hnsd seeds: 9/56 unique IPs accepted TCP/44806
  • PR Update seeds from hnsnodes #134 seeds: 19/21 unique IPs accepted TCP/44806
  • all three 103.152.197.x Nathan nodes accepted TCP/44806

I hit one build issue: the final seed entry is missing a trailing comma before NULL in src/seeds.h.

After adding that comma locally, the branch built successfully, ./test_hnsd passed, and a runtime smoke test started, loaded the new seeds, injected the checkpoint, and began advancing headers.

This removes inaccessible peers and adds new peers found using hnsnodes
@shadstoneofficial

Copy link
Copy Markdown

Retested after the comma fix. Looks good from my side.

Fresh clone of commit 7f9e453 built successfully on macOS under Rosetta/x86_64 with no local patches:

  • ./autogen.sh passed
  • ./configure passed
  • make passed
  • ./test_hnsd passed

Seed reachability from my network in Bangkok also looks much better:

  • current master hnsd seeds: 9/56 unique IPs accepted TCP/44806
  • PR Update seeds from hnsnodes #134 seeds: 19/21 unique IPs accepted TCP/44806
  • all three 103.152.197.x Nathan nodes accepted TCP/44806

I also ran a runtime smoke test on high local DNS ports. hnsd started, loaded the new seeds, injected the checkpoint at height 136000, and began advancing headers.

LGTM from my testing.

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.

2 participants