-
Notifications
You must be signed in to change notification settings - Fork 3
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 patterns to OIDC identity mapping #147
Conversation
Add validation to 'username' attribute to ensure 'scope' attribute is also Don't set 'scope' attribute unless it's not empty string Update documentation to include the same
Optional: | ||
|
||
- `audience` (String) Sets of (space separated) the JFrog services to which the mapping applies. Default value is `*@*`, which applies to all services. | ||
- `expires_in` (Number) Token expiry time in seconds. Default value is 60. | ||
- `groups_pattern` (String) Provide a pattern which is used to map OIDC groups to Artifactory groups. |
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.
Should you add an example here? It should be inside of {{}}
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.
I added example in the section above.
- `username` (String) User name of the OIDC user. Not applicable when `scope` is set to `applied-permissions/groups`. Must be set when `scope` is set to `applied-permissions/roles`. | ||
- `username_pattern` (String) Provide a pattern which is used to map OIDC user to Artifactory 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.
Should you add an example here? It should be inside of {{}}
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.
Same, look at the TF example
Closes #145