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

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

Details

Reviewers

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

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 ↗	(On Diff #31828)	dnsseed -> bitcoin-seeder
51 ↗	(On Diff #31828)	dito

This revision now requires changes to proceed.Jan 17 2022, 16:42

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

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

Path

Size

src/

seeder/

README.md

20 lines

Diff 31835

View Options

[seeder] Add some useful information about testing in the READMEClosedPublicActions