search

hlandau / madns

:globe_with_meridians: Authoritative DNS server engine for Go
10 stars 1 forks source link

Add stream isolation via EDNS0 #4

Closed JeremyRand closed 4 years ago

JeremyRand commented 5 years ago

This is an API-breaking change.

JeremyRand commented 4 years ago

@hlandau This is ready for review. You can test it by building ncprop279 with the following PR's applied in addition to this one:

https://github.com/namecoin/ncdns/pull/97 https://github.com/namecoin/ncprop279/pull/9

Typing the following into ncprop279's STDIN should work fine as before:

RESOLVE 1 domob.bit

Typing the following should yield an error about the stream isolation ID not being passed to namecoind:

RESOLVE 1 domob.bit FOO

(Obviously, that error will be replaced later with functionality that passes the stream isolation ID to namecoind, but that's out of scope for this PR, and is blocked on a Conformal PR.)

hlandau commented 4 years ago

v2.0.0 tagged (breaking API change). @JeremyRand