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

Add fetching of FABRID policy descriptions to the SCION CLI. #174

Open
wants to merge 4 commits into
base: scionlab
Choose a base branch
from

Conversation

jeltevanbommel
Copy link

@jeltevanbommel jeltevanbommel commented Dec 18, 2024

This PR adds the option to fetch policy descriptions to the scion CLI tool. Largely based on jeltevanbommel#5

Concretely the CLI dials the local CS using the local SCION daemon. Then

  • for a requested FABRID Local policy: the local CS checks whether the policy is already known locally (either because it is a policy it owns, or it has recently requested the same identifier and it is cached) and returns the description. If it is not known to the local CS, the local CS dials the remote AS that has announced the FABRID Local policy identifier and requests the description.
  • for a requested FABRID global policy: the local SCION Daemon uses the global store of FABRID policies, which is configurable in the SCION Daemon configuration file, to request the policy description.

Other features:

  • Pretty printing the result
  • Add a special case for when we request a remote policy and it is a policy from our own AS.

Usage:

bin/scion fabrid G1100 --sciond $(./scion.sh sciond-addr 112)
bin/scion fabrid L1121 1-ff00:0:112 --sciond $(./scion.sh sciond-addr 112) --format human
bin/scion fabrid L1121 1-ff00:0:112 --sciond $(./scion.sh sciond-addr 112) --format json

Depends on #173


This change is Reviewable

…t, rename variable that shadows package name
Pretty print the results. Also add a special-case in case we request a remote policy and it is a policy from our own AS.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant