-
Notifications
You must be signed in to change notification settings - Fork 2
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 OIDC support #47
Add OIDC support #47
Conversation
Add OIDC identity mapping import
Update documentation
Switch it from object type to plain string type and let users use encodejson() to define the correct JSON.
Required: | ||
|
||
- `scope` (String) Scope of the token. You can use `applied-permissions/user`, `applied-permissions/admin`, or `applied-permissions/group`. | ||
- `username` (String) User name of the OIDC user. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this PR!
At present in JFrog UI it is possible to specify a list of groups when applied-permissions/group
is selected, e.g. see JFrog CLI GitHub Action configuration. It would be really nice to have it here as well so it is possible to utilize access tokens for Non-existing Users feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@oxdef Thanks for the suggestion. I'll update the PR today with support for groups.
Make 'priority' a required attribute Make 'username' an optional attribute Improve documentation
Fix typo in test name
Close #26, #29, #31, #38
platform_oidc_configuration
andplatform_oidc_identity_mapping
resources.