Skip to content
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

[FEATURE] Add DNS resolver to acquire seed peers #382

Open
alexeykiselev opened this issue Nov 3, 2020 · 0 comments
Open

[FEATURE] Add DNS resolver to acquire seed peers #382

alexeykiselev opened this issue Nov 3, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@alexeykiselev
Copy link
Member

Аbstract
A gowaves node knows IP addresses of seed nodes (initial nodes to connect to) from the built-in configuration. Also the initial nodes could be provided by the command line parameter -peers. It seems better to replace or the built-in seed peers with the DNS record lookup.

Specification
Built-in configuration should support domain names along with seed nodes IP addresses. Those domain names should be resolved to underlying IP addresses and those addresses should be treated as usual seed nodes IP addresses.
The argument of -peers command line parameter should be parsed the same way as built-in configuration.

Backwards Compatibility
Configuration and -peers command line parameter should support IP addresses passing for backward compatibility.

Examples and Implementation
Bitcoin operates the same way.
https://github.com/btcsuite/btcd/blob/535f25593d47297f2c7f27fac7725c3b9b05727d/connmgr/seed.go#L33

@alexeykiselev alexeykiselev added the enhancement New feature or request label Nov 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant