Skip to content

TrustStore: Implement recurser #3222

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 4, 2019
Merged

TrustStore: Implement recurser #3222

merged 1 commit into from
Oct 4, 2019

Conversation

oncilla
Copy link
Contributor

@oncilla oncilla commented Oct 4, 2019 •
edited by kormat
Loading

Adds:

  • Recurser that decides whether recursion is allowed for a given client.
    • ASLocalRecurser allows all clients in the local AS.
    • LocalOnlyRecurser allows only the local application.

This change is Reviewable

@oncilla oncilla added c/tooling SCION network tools c/CPPKI SCION Control-plane PKI labels Oct 4, 2019
@oncilla oncilla added this to the Q4S1 milestone Oct 4, 2019
@oncilla oncilla requested a review from scrye October 4, 2019 10:38
@oncilla oncilla removed the c/tooling SCION network tools label Oct 4, 2019
scrye
scrye approved these changes Oct 4, 2019
View reviewed changes
Copy link
Contributor

@scrye scrye 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

@oncilla
TrustStore: Implement recurser
9e5fef8 
Adds:
- Recurser that decides whether recursion is allowed for a given client.
  - ASLocalRecurser allows all clients in the local AS.
  - LocalOnlyRecurser allows only the local application.
@oncilla oncilla merged commit a09f0be into scionproto:master Oct 4, 2019
@oncilla oncilla deleted the pub-recurser branch October 4, 2019 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@scrye scrye scrye approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
c/CPPKI SCION Control-plane PKI
Projects
None yet
Milestone
Q4S1
Development

Successfully merging this pull request may close these issues.
2 participants
@oncilla @scrye