-
Notifications
You must be signed in to change notification settings - Fork 39
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
DOC-1011 Add Keycloak and AzureAD to AuthZ topic for Console #975
Conversation
✅ Deploy Preview for redpanda-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
| `user` | ||
| user | ||
| AzureAD | ||
| User's OID (or email address if using a custom identifying claim) |
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 we write OID out in full first time, assuming this is the object ID?
| Google Group Name (which is an email address) | ||
|
||
| `user` | ||
| user | ||
| GitHub | ||
| Login handle / GitHub username |
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.
Here we use spacing before and after forward slash but not on line 162
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.
Looks good. Just some suggestions but no blockers.
Co-authored-by: Angela Simms <[email protected]>
Description
Resolves https://github.com/redpanda-data/documentation-private/issues/
Review deadline:
Page previews
https://deploy-preview-975--redpanda-docs-preview.netlify.app/current/console/config/security/authorization/#subjects
Checks