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

Update AAD auth doc #3890

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ryanelliottsmith
Copy link
Contributor

Include note on the fact that the user that is used to test authentication is bound to the 'admin' account.

For Rancher (product) docs only

When contributing to docs, please update the versioned docs. For example, the docs in the v2.6 folder of the rancher folder.

Doc versions older than the latest minor version should only be updated to fix inaccuracies or make minor updates as necessary. The majority of new content should be added to the folder for the latest minor version.

Include note on the fact that the user that is used to test authentication is bound to the 'admin' account.
@btat btat requested a review from cbron May 11, 2022 22:03
> **Note:**
>
> The AD user pertaining to the credentials used to test will be mapped to the local principal account and assigned administrator privileges in Rancher. You should therefore make a conscious decision on which account you use to perform this step.

Copy link
Contributor

@cbron cbron May 12, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add this to all versions of the doc, 2.6, 2.4 etc..

Also, should this be done at a generic level for all oauth/ldap providers, I would think so.

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.

2 participants