Page MenuHomePhabricator
Differential D10834

[seeder] Add some useful information about testing in the README
ClosedPublic
Actions

Authored by thonkle on Jan 17 2022, 15:35.

Details

Reviewers
Fabien
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABC533b749aced8: [seeder] Add some useful information about testing in the README
Summary

I found this to be helpful. Not being a DNS expert, I couldn't be sure if the seeder was not producing output or if my DNS records were broken or if my iptables were configured incorrectly. My confidence has improved now that I know how to eliminate the first case consistently. I think others will find this useful too.

Backport of my upstream change here: https://github.com/sipa/bitcoin-seeder/pull/96

Test Plan
./bitcoin-seeder -host=dnsseed.example.com -ns=localhost -mbox=thonkle@protonmail.com -port=15353
dig @:: -p 15353 dnsseed.example.com

Diff Detail

Event Timeline

thonkle created this revision.Jan 17 2022, 15:35
Owners added a reviewer: Restricted Owners Package.Jan 17 2022, 15:35
Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 17 2022, 15:35
thonkle requested review of this revision.Jan 17 2022, 15:35
Harbormaster completed remote builds in B17926: Diff 31828.Jan 17 2022, 15:47
Fabien requested changes to this revision.Jan 17 2022, 16:42
Fabien added a subscriber: Fabien.
Fabien added inline comments.
src/seeder/README.md
44

dnsseed -> bitcoin-seeder

51

dito

This revision now requires changes to proceed.Jan 17 2022, 16:42
thonkle updated this revision to Diff 31831.Jan 18 2022, 14:54

Renamed dnsseed -> bitcoin-seeder. There were two other places in the README that also have not been updated.

Harbormaster completed remote builds in B17929: Diff 31831.Jan 18 2022, 15:10
Fabien accepted this revision.Jan 18 2022, 16:33

You meant have BEEN updated I guess ?

This revision is now accepted and ready to land.Jan 18 2022, 16:33
Closed by commit rABC533b749aced8: [seeder] Add some useful information about testing in the README (authored by thonkle). · Explain WhyJan 18 2022, 16:37
This revision was automatically updated to reflect the committed changes.
thonkle added a commit: rABC533b749aced8: [seeder] Add some useful information about testing in the README.

Revision Contents
Changeset List

PathSize
src/
seeder/
16 lines

Diff 31828

View Options

src/seeder/README.md

Loading...