-
Notifications
You must be signed in to change notification settings - Fork 111
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 DNS #7
Comments
This format in ambiguous regarding IPv4/IPv6. Other options:
All of these would roughly resolve to |
also, which record? just A? or allow TXT to work too:
(hard to do defaults...) (update: ahh i see you mentioned |
i like maybe: /dns/baz.org/ip # either 4 or 6
/dns/baz.org/ip4 # ipv4
/dns/baz.org/ip4 # ipv6 |
yeah that's what i was after :) A full example:
It'd break with the quasi-rule that a protocol is a 2-tuple, but we can say that you MUST know how many segments follow once you see the first one -- e.g. you see |
@lgierth that sounds good to me
Yep, i think that's fine too (deterministic) |
i also havent seen other multiaddr impls, so easy to change for us |
@lgierth we should probably support record types here.
? |
Do you mean for SRV/TXT/etc. records? Cause A and AAAA always implicate ip4/ip6. |
ah right 👍 |
Some comment i got by email is missing. for posterity:
Yes you should use the |
That's my comment. Thought it was a little off topic so I deleted it 🐰 But since you mention it, I'll have you know that I got |
Yes we'll have to make a few changes to go-multiaddr, namely support for triplets, and some kind of resolution step that turns |
The problem with "resolving |
Your application code would be working with |
Well, I have my case solved, it doesn't matter to me much. But I do like (and did in my reimplementation) manet to handle connection differences between EDIT: I'm just trying to seduce you with the neatness of this, and saying that it's possible and available. Not asking for anything. |
|
Closing this in favor of multiformats/multiaddr#22 |
Multiaddrs like these should work:
The text was updated successfully, but these errors were encountered: