-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Support storing NEO addresses in NNS contract #2284
Comments
I thought about this problem. But I find it a bit strange to bind NEO addresses directly to domain names. It would be nice to bind to an email address. |
I think that precisely a named service in blockchain it's for store this information. |
I found some scenarios, such as Dapp domain name can be resolved into Neo address. We should propose a standard that uses TXT records to store addresses.
This way we don't need to modify the core. |
TXT can be anything, adding a proper type is easy and it will help us ensure that the thing in this record is actually a valid address. |
Please check neo-project/proposals#133 |
Look at it from the perspective of NNS user. I want to know if
This TXT thing is good for independent experimental features, but NEO addresses seem like a core concept to me. |
But these steps are done by the wallet or client, the users don‘t need to perceive it. |
My design philosophy is to design a lightweight core as much as possible. For problems that can be solved with standards, try not to add functions to the core. Unless this can bring very obvious advantages, such as performance improvements. |
Following this logic we might as well strip it down to TXT and CNAME only (even to TXT only with a little more effort). Yet we still have A and AAAA. And for blockchain use case NEO record type (or its TXT equivalent) is expected to be used more frequently than AAAA and probably even A, just because that's what people care about when they're using Neo. A and AAAA records don't make their $BROWSER suddenly use this data, while NEO addresses are expected to be supported by every wallet implementation. So it'll sure work with TXT (and neo-project/proposals#133 is useful for other purposes), but it doesn't look like an optimal choice for addresses to me. |
NNS is the domain name service on neo. The main application scenarios of |
I'm not sure I'm convinced, but probably the main thing here is to have this use case covered one way or another and as neo-project/proposals#133 does this, so be it, we just need to add support for it everywhere. |
Summary or problem description
NNS currently defines records of types A, CNAME, TXT and AAAA for names which roughly mimic regular DNS types. So we can resolve some name into IP (v4 or v6) or opaque text record effectively. What we can't do is resolve it into some NEO address which I think can be useful for "send some GAS to
whatever.wallet
" scenarios wherewhatever.wallet
can be resolved via NNS into someNUVPACMnKFhpuHjsRjhUvXz1XhqfGZYVtY
NEO address. See #1179 and #1450 also.Do you have any solution you want to propose?
Add "NEO" record type to store (syntactically checked) NEO addresses.
Neo Version
Where in the software does this update applies to?
The text was updated successfully, but these errors were encountered: