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

scrpto: Move KeyMeta + Version #2912

Merged
merged 3 commits into from
Jul 25, 2019
Merged

Conversation

oncilla
Copy link
Contributor

@oncilla oncilla commented Jul 25, 2019

Previously, the TRC package defined its own KeyMeta and Version format.
However, this will be reused by the issuer and AS certificates.

This PR moves the definitions to scrypto.

Additionally, fixes #2911


This change is Reviewable

Previously, the TRC package defined its own KeyMeta and Version format.
However, this will be reused by the issuer and AS certificates.

This commit moves the definitions to scrypto.
@oncilla oncilla added the c/CPPKI SCION Control-plane PKI label Jul 25, 2019
@oncilla oncilla added this to the Q3S1 milestone Jul 25, 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 18 of 18 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@oncilla oncilla merged commit 78c5b86 into scionproto:master Jul 25, 2019
@oncilla oncilla deleted the pub-move-key-meta branch July 25, 2019 13:11
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.

scrypto: Validity period should disallow unknown fields
2 participants