-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
EIP-634: Storage of text records in ENS #2439
Comments
There seem to be support for this resolver in ENS already. Is this widely supported? If it is, it would make sense trying to bring this ERC to the Final state. I'd be interested in it for eip-automerger/automerger#7 |
I added and deployed this as the public resolver (resolver.eth) about 2.5 years ago, and it has been in every public resolver since, so I think we're good in terms of adoption. :) |
Someone suggested a list of additional fields somewhere, but I cannot find it now. Some come to mind from that list and I will add it here. I think it makes sense to keep this one comment up to date with others' suggestions too. These may make more sense to expand into a separate EIP.
Notes:
|
Why not include them in the EIP? |
@axic, I was thinking they should be up for discussion before including them in case someone has objections? Or in case there is a better standard I don't know about, for example to encode a GPS location or an ISO-XXXX for locations, addresses, etc. |
For phone number, please specify E.164 formatting. You can edit the above example to: +14165551496 |
Thinking out loud... What about something for a label's canonical case used for display logic? It would need to be enforced (by the client) that the
|
@ricmoo I think it sounds looks a nice idea. |
Shouldn't |
I love this. What would we need to do to merge this to the EIP? Wondering if this got stopped due to some concerns articulated elsewhere @ricmoo |
Oh, no. It’s still on-going. This thread is for discussions about new fields. I just want more community feedback before committing anything. :) |
@ricmoo May I propose the following additional keys: mail: a mailing address Is there a good reason why we shouldn't just add standard keys for the handles of the top 20 social networks/chat apps? |
I've created a draft suggestion (just didn't push the PR) and included these: +- **vnd.gitlab** - a GitLab username
+- **vnd.peepeth** - a Peepeth username
+- **vnd.twitter** - a Twitter username
+- **vnd.keybase** - a Keybase username
+- **vnd.reddit** - a Reddit username So I'd second @brantlymillegan's keybase suggestion. However I would say PGP doesn't fit in here, but should have its own specification for both fingerprint and public key. Reason: it takes less space storing it as a binary, than its text serialisation. |
@axic Thanks. And yes, I agree re PGP. |
Should the EIP specific an aspect ratio for avatars, e.g. 1 x 1? |
Name: name.first, name.last, name.middle, name.prefix, name.suffix, name.nickname |
Heya! Sorry for the delay... There was some discussion (I think on Twitter), about using reverse dot notation for vendor keys. I think the ones already suggested should remain (e.g. vnd.twitter), but going forward any service may use their domain name as their key, without any need to update this EIP. For some examples:
Of course, each service can decide how they want their hierarchal namespace broken up. Perhaps keybase simply wants Also, a service can use any fully-qualified name they own. So, peepeth may decide to use What do people think? I will take a look and merge the above suggestions that don't conflict with this one next week and depending on what the discussions lead to, include the rest too. |
Why retain |
I do agree, except things are already using it. Maybe there could be a “applications SHOULD check for Although, maybe it is enough to coordinate with the ENS app, which I think other than maybe ethers CLI is the only thing I know of that uses it? |
@wpmonty sounds good. In addition, the |
For standards, I recommend thinking about 5-10 years in the future rather than right now. If the standard you want is |
I tend to agree. Also, I'd bet very few people or dapps (if any) are making use of this right now. Not sure how hard it would be for someone to run a check? |
Sounds perfect. I'll just make a backwards compatibility note. :) |
Sorry to jump into the conversation so late. I think TextResolver is very interesting for many use cases where I am involved. Have you considered using for the names of standard keys the vocabulary of Schema.org? It is used heavily in other environments and it defines not only the names of many attributes for many entities, but also in many instances the semantics involved and the specific format for the values. For example, here you have the attributes for Person (https://schema.org/Person), including of course name, address, birthDate, etc. Another example is GeoCoordinates (https://schema.org/GeoCoordinates), where if you use latitude and longitude, the spec requires to use WGS 84 for their values. This may reduce creeping of vendor keys for many use cases where they want essentially the same entity. Even in the cases where the entity is not in Schema.org, there is a defined process to extend the vocabulary, either to the base vocabulary or creating and industry-specific extension. For example, see here for the Automotive extension: https://schema.org/docs/automotive.html and here for Banks and Financial Institutions: https://schema.org/docs/financial.html. |
This was never answered, and now that I'm actually using ENS a bit more I do feel pretty strongly that this should be the case. I believe we should really be building things to be censorship resistant by default, and part of that means making choices that make censorship resistance the easiest path for integrators to take. By having avatar be a URL, you can use As a prime example, the ENS manager app itself doesn't support |
This is another reason I need to update it to allow dotted notation. There is often nested data you want to include, and you could imagine a solution similar to how namecoin's website did it back in the day, which is to allow a "proof" field. So, for example, I want a I have earmarked some time this week to catch up on this EIP and a bunch of other things I've been putting of way too long. Would love to get some active discussions this week. |
Yeah this was very very non-intuitive when setting up. Essentially every application of said notation uses Also, regarding PGP keys and their size, it may make more sense to add your PGP fingerprint and make use of a keyserver? Either query existing keyservers or have a dedicated one for .eth domains. |
@ricmoo I propose default text record keys for:
|
There has been no activity on this issue for two months. It will be closed in a week if no further activity occurs. If you would like to move this EIP forward, please respond to any outstanding feedback or add a comment indicating that you have addressed all required feedback and are ready for a review. |
I am considering moving this EIP to withdrawn, once we have a similar repo to EIPs for ENS, citing the reason that it is being moved to that repo. Does this sound good? |
There has been no activity on this issue for two months. It will be closed in a week if no further activity occurs. If you would like to move this EIP forward, please respond to any outstanding feedback or add a comment indicating that you have addressed all required feedback and are ready for a review. |
There has been no activity on this issue for six months. It will be closed in a week if no further activity occurs. If you would like to move this EIP forward, please respond to any outstanding feedback or add a comment indicating that you have addressed all required feedback and are ready for a review. |
@brantlymillegan I think there should be a placement option for avatars. Something like this
Similar to how avatars work on social media and web/mobile services: when a user selects a picture for an avatar, she can move the cropping window. Here are the examples from my Facebook and Twitter: Screen.Recording.2022-09-30.at.13.20.42.movWhat worries me is that some heads can be cut in half and the users can’t do anything. |
Another perhaps less pressing problem might be that some 👥 artists and 👥 designers might not be happy about cropping their arwork at all. So it might make sence to consider adding an option to fit the artwork into a square:
I think this feature might play well together with ENS background gradients @brantlymillegan what do you think? |
Sorry to jump into this discussion. With the confusion "Why not all text records are content hash rather than a URL? Or otherwise why not EIP-1577 use a URL just like other text records, rather than a content hash?" in my mind, I browsed and came to this discussion seeing your explanation. It seems content hash is used to make the resource identification by design in a censorship-resistant approach. But I am wondering if the content hash is designed to exclude all non-censorhsip-resistant protocols? E.g. if having avatar be content hash, will an HTTPS URL encoded in multiformat be supported? Or only "hash" related protocol(ipfs/arweave...) supported? |
Why not a multiaddr? |
A discussion thread has been opened for this proposal: https://ethereum-magicians.org/t/eip-634-storage-of-text-records-in-ens/14138/2 |
This is a discussions-to thread (as per EIP-1) for: EIP-634 Storage of text records in ENS.
This topic section may be updated occasionally to reflect additional information which becomes relevant to interested parties.
The text was updated successfully, but these errors were encountered: