raybellis / draft-bellis-dnsop-xpf

DNS X-Proxied-For
0 stars 1 forks source link

use IANA family numbers? #9

Closed Habbie closed 7 years ago

Habbie commented 7 years ago

We use '4' and '6' to indicate v4 and v6. These numbers do not appear like that in any registry. edns-client-subnet and clientid use https://www.iana.org/assignments/address-family-numbers/address-family-numbers.xhtml. @arjenz noticed during wireshark implementation that our deviation makes his code longer as he cannot rely on the already present IANA list.

Habbie commented 7 years ago

IANA family numbers are 16 bits, this costs us our 4 bits of extensibility but we could use 65535 to signal some future extension perhaps.

Habbie commented 7 years ago

These numbers do not appear like that in any registry

I lied - https://www.iana.org/assignments/version-numbers/version-numbers.xhtml

Habbie commented 7 years ago

So, the argument is somewhat weak but I'd still like to make a conscious decision here.

raybellis commented 7 years ago

The 4 bit field in XPF indicates the L3 transport protocol version, but we only support IP anyway.

Habbie commented 7 years ago

Moved the QUIC stuff to #10

arjenz commented 7 years ago

Implemented lookup for RFC2780 / https://www.iana.org/assignments/version-numbers/version-numbers.xhtml in the Wireshark dissector, https://code.wireshark.org/review/#/c/22803/ So the only argument left now is being in sync with edns-client-subnet and clientid :)

raybellis commented 7 years ago

On 26/07/2017 14:19, Arjen Zonneveld wrote:

So the only argument left now is being in sync with edns-client-subnet and clientid :)

There's no argument - EDNS Client Subnet used the wrong one ;-)

Since clients only access DNS over IP, and prefix lengths have no semantic meaning in some of the non-IP family AF_XXX protocols, why use a registry with illegal values when the only sane ones are AF_INET and AF_INET6.

Habbie commented 7 years ago

Alright, settled!

raybellis commented 7 years ago

BTW, the IANA Address Family Registry and the BSD socket AF_INET and AF_INET6 macros don't even use the same values.

Habbie commented 7 years ago

BTW, the IANA Address Family Registry and the BSD socket AF_INET and AF_INET6 macros don't even use the same values.

I noticed that when getnameinfo refused to convert an IP in AF 4 ;)