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

Add option to advertise relay addresses along with other addresses #4992

Open
whyrusleeping opened this issue May 2, 2018 · 2 comments
Open
Labels
help wanted Seeking public contribution on this issue

Comments

@whyrusleeping
Copy link
Member

Currently, if we want to advertise relay addresses, we have to specify all the addresses we want to advertise in the Addresses.Announce config field (because that field correctly replaces all addresses instead of augmenting them). We should have a way to either say "Add relay addresses to my list of addresses that I announce!" or "add this set of addresses to the list of addresses that I announce".
I'm leaning towards the relay specific option, but opinions are welcome.

@whyrusleeping whyrusleeping added the help wanted Seeking public contribution on this issue label May 2, 2018
@grokcoder
Copy link
Contributor

Hello, I am glad to help this issue. Do you want a more conveniont way to manipulate the relay addrs?
How about to add some command like

ipfs announce list // list the announce addr
ipfs announce add <addr>
ipfs announce rm <addr>

@bigs
Copy link
Contributor

bigs commented May 21, 2018

@grokcoder @whyrusleeping i can get started on this tomorrow. i'd likewise favor the specific option since the less specific option can be built atop it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Seeking public contribution on this issue
Projects
None yet
Development

No branches or pull requests

3 participants