-
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Proof of concept based on Rotonda. #6
Draft
ximon18
wants to merge
21
commits into
main
Choose a base branch
from
demo-with-rotonda-base
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
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
… Rotonda based component system for handling zone receipt, preview, signing and serving. Uses a custom branch of `domain` to resolve difficulties encountered implementing the `Service` trait.
…. Lacks sending an event to CC on approval.
…e signer to sign the zone.
… request error handling and logging.
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 draft PR contains a proof-of-concept "bump-in-the-wire" DNSSEC signing server that can receive and publish zones via XFR.
TL;DR
See
README.md
andnameshed.conf
for more information.The longer explanation :-)
It is based on a modified branch of
domain
to address some issues found withService
trait bound matching and to address a short coming in theZoneUpdater
.It is not meant to be fast or memory efficient or even a correct DNS server (as domain lacks various things needed for that such as setting the AA flag correctly on responses, support for various record types is missing, and support for DNSSEC signed responses). It is also not meant to be clean code without bugs.
Instead it is meant as a quickly put together demonstration of the design ideas that were discussed internally about a component based model with events flowing to a Central Command and commands in turn being issued to components. It doesn't strictly implement the async fn event and command model that was discussed, as only events are communicated via async fn calls while commands are issued via multiple-producer single-consumer channel.
Known issues include blocking the HTTP API while signing.
Zone
s VIA COLLECTION FIRST INTO A VEC.todo!()
in the code even though all the necessary puzzle pieces exist.Missing features include:
I probably forgot a lot of things... ;-)