-
Notifications
You must be signed in to change notification settings - Fork 291
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
multi: Add HTTP seeder support. #2017
Conversation
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.
In addition to the inline comments, this is also failing the tests.
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.
The third commit needs to be amended to conform to the code contribution guidelines and I would also suggest updating the commit the deprecates nodnsseed
to also deprecate the chaincfg.Params.DNSSeeds
field since it will need to be removed in the next major version bump of chaincfg
.
Seeder identifies the hostname as a seeder.
connmgr: deprecate SeedFromDNS config: deprecate nodnsseed; add noseeders
I added a debug statement to ensure the returned values match the query string as expected and verified that it works as expected over Tor as well. |
Requires decred/dcrseeder#25