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

[FR] - DNS TXT record support for pool metadata json #1354

Closed
glitch40 opened this issue Jun 28, 2020 · 6 comments
Closed

[FR] - DNS TXT record support for pool metadata json #1354

glitch40 opened this issue Jun 28, 2020 · 6 comments
Assignees
Labels
cli revision is used to group all the Node CLI tickets enhancement New feature or request
Milestone

Comments

@glitch40
Copy link

Exernal

Support added for Pool metadata from Domain DNS TXT record

Instead of defined url support in pool registration that is restricted by character length add the ability for the meta data to be pulled directly from DNS TXT records, Instead of unsecured random urls this would enable pools using 3rd party hosts for websites the ability to update and maintain the pool metadata directly on the domain registered for the pool this would add an additional security layer to verify the ownership of the domain and the meta data for pool registration.

Example anonaf.com

dig -t txt anonaf.com

;; ANSWER SECTION:
anonaf.com. 3600 IN TXT "{ name: TestPool, description: The pool that tests all the pools, ticker: TEST, homepage: https://teststakepool.com }"

Alternatives would be self hosting meta data on one of the relays.

Currently hosting sites like Wix or other 3rd party sites have extremely long urls that do not work with the current system of hosting (64 character limit) the json on a url requiring another work around of a url shortener to resolve this issue or running a web server from one of the relays to host the json. The url shortners or third party file upload sites being used to host the metadata could expire in the future leading to even more issues in the future when the meta data can no longer be found where as the DNS TXT record will always be in the same place so long as the domain is registered.

@glitch40 glitch40 added the enhancement New feature or request label Jun 28, 2020
@kevinhammond kevinhammond added this to the Backlog milestone Jun 29, 2020
@kevinhammond kevinhammond added the cli revision is used to group all the Node CLI tickets label Jun 29, 2020
@andreacfromtheapp
Copy link

+1 , this would be awesome and less prone to spoofs.

@moomanchoo
Copy link

+1

@vix-io
Copy link
Contributor

vix-io commented Sep 15, 2020

Please put this as a suggestion in the CIP: https://github.com/cardano-foundation/CIPs/issues/new

@vix-io vix-io closed this as completed Sep 15, 2020
@glitch40
Copy link
Author

Please put this as a suggestion in the CIP: cardano-foundation/CIPs#15

Hello not sure why this was closed. This is about securely hosting the metadata as an alternative to the current solution and providing a proof of ownership to the pool tied to domain not including metadata or extended metadata structure per the CIP referenced.

@rdlrt
Copy link

rdlrt commented Sep 15, 2020

@vix-io - I have to agree, Please do not rush through closing issues. The referred CIP is quite different and talks about content of linked JSON - not pulling it from DNS TXT sources

@dcoutts
Copy link
Contributor

dcoutts commented Sep 15, 2020

The point is, you may find that CIP is an adequate solution to the problem, or if not you can either advocate for an alternative mechanism within that CIP, or craft a wholly new CIP.

Either way, the CIP process is going to be the better avenue for this kind of feature request & design review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli revision is used to group all the Node CLI tickets enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

8 participants