Add stream isolation via EDNS0 #4
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
JeremyRand
force-pushed
the
stream-isolation
branch
from
43ddead
to
e5aa764
Compare
JeremyRand
force-pushed
the
stream-isolation
branch
from
e5aa764
to
dfce7bd
Compare
5 tasks
This is an API-breaking change.
JeremyRand
force-pushed
the
stream-isolation
branch
from
dfce7bd
to
a27bbcb
Compare
JeremyRand
changed the title
|
@hlandau This is ready for review. You can test it by building ncprop279 with the following PR's applied in addition to this one: namecoin/ncdns#97 Typing the following into ncprop279's STDIN should work fine as before: Typing the following should yield an error about the stream isolation ID not being passed to namecoind: (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.) |
5 tasks
|
v2.0.0 tagged (breaking API change). @JeremyRand |
JeremyRand
added a commit
to namecoin/ncdns
that referenced
this pull request
Oct 19, 2019
06d6efb backend: Stream-isolate LRU cache (JeremyRand) ed8fa68 Add plumbing for stream isolation (JeremyRand) Pull request description: This PR adds the plumbing for stream isolation (from madns to namecoin, without passing the stream ID to namecoind), and stream-isolates the LRU cache. A follow-up PR will pass the stream ID to namecoind; that's currently blocked on #103 . Prerequisites: - [x] Merge hlandau/madns#4 - [x] Merge hlandau/madns#5 - [x] Tag a madns release (with major version 2). - [x] Replace `madns.v1` dependency with `madns.v2`. - [x] Tag a madns release (with hlandau/madns#5 included). ACKs for commit 06d6ef: Tree-SHA512: ee385454aef3b802899659db74bbeb28ec6555216f4a4ccfc6047c08f32272080b7f76f851e4b291204ab41525bbfae30e7cdc663b49595d7d5105744810c763
JeremyRand
added a commit
to namecoin/ncprop279
that referenced
this pull request
Oct 19, 2019
d49e2ed Add EDNS0 stream isolation (JeremyRand) Pull request description: Fixes #8 Prerequisites: - [x] Merge hlandau/madns#4 - [x] Merge hlandau/madns#5 - [x] Tag a madns release (with major version 2). - [x] Replace `madns.v1` dependency with `madns.v2`. - [x] Tag a madns release (with hlandau/madns#5 included). ACKs for commit d49e2e: Tree-SHA512: 540ccb75358b7dbe640f88151d6bee45d625d22aa58558054c81f1f08412d4bfa43a9cb2730a59898d3e4ff0da1c62a610ac266877419a311b4b89ca5dc87b8e
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is an API-breaking change.