Skip to content
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

keyconf: Add key ring #3528

Merged
merged 1 commit into from
Dec 17, 2019
Merged

Conversation

oncilla
Copy link
Contributor

@oncilla oncilla commented Dec 16, 2019

Add simple key ring that loads the key from the file system on demand.
This implements the trust.KeyRing interface.


This change is Reviewable

@oncilla oncilla added the c/CPPKI SCION Control-plane PKI label Dec 16, 2019
@oncilla oncilla added this to the Q4S5.D milestone Dec 16, 2019
Copy link
Collaborator

@lukedirtwalker lukedirtwalker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 3 of 3 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

Add simple key ring that loads the key from the file system on demand.
This implements the trust.KeyRing interface.
@oncilla oncilla merged commit 7a09ecc into scionproto:master Dec 17, 2019
@oncilla oncilla deleted the pub-trust-key-ring branch August 7, 2020 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c/CPPKI SCION Control-plane PKI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants