Skip to content

Adds secret information to the Principal #50

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

Closed
wants to merge 2 commits into from
Closed

Adds secret information to the Principal #50

wants to merge 2 commits into from

Conversation

casusbelli
Copy link

This extends the Principal message for storing authentication information from the CO.

This change depends on PR #30 and was spun off from a discussion in that PR.

This change adds an optional principal in
order to enable applying basic authorization control,
e.g. for shared filesystem backends. This is
done by mapping all volume access through a volume
volume publication to the principals ids.
Further details in issue #17 .
@jdef
Copy link
Member

jdef commented Sep 7, 2017

Credentials have landed in #83. Is this PR still needed?

@jdef jdef added the lifecycle/stale PR has been inactive for 90+ days. label Oct 4, 2018
@saad-ali saad-ali added lifecycle/rotten Stale PR has been inactive for 30+ days. and removed lifecycle/stale PR has been inactive for 90+ days. labels Nov 7, 2018
@saad-ali
Copy link
Member

saad-ali commented Nov 7, 2018

@saad-ali
Copy link
Member

saad-ali commented Mar 6, 2019

@saad-ali saad-ali closed this Mar 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lifecycle/rotten Stale PR has been inactive for 30+ days.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants