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

docs: add other required fields to user setup for clustered #5639

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

david-rusnak
Copy link
Contributor

Hey team! A customer attempted to use azure docs during clustered set up and ran into an issue with the current code example for setting azure users in the appInstance. I've added the rest of the fields as required (similar to keycloak).

I see there is this PR out for reorganizing the clustered docs, so maybe it can be done there to avoid conflicts? Let me know the best way forward, thanks!

Copy link
Collaborator

@sanderson sanderson left a comment

Choose a reason for hiding this comment

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

Thanks @david-rusnak! I'll take care of porting these over to the other PR.

@sanderson sanderson merged commit 6532ad0 into master Oct 9, 2024
1 check passed
@david-rusnak
Copy link
Contributor Author

@sanderson Hey Scott, just a note on this that @eatondustin1 brought up to me. While it's true that the email can be arbitrary for auth purposes, for any auditing the customer wants to do - they should probably use a valid email/identifier so any actions that track the user who initiated it can be audited later.

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