-
Notifications
You must be signed in to change notification settings - Fork 1.1k
docs: add OIDC documentation for Microsoft Entra ID user auth #20202
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
Conversation
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA. |
|
recheck |
|
everything should be tip-top shape, man tight checks on those PRs! |
|
Missed a few spots with "Google" lol, used the google.md as a boilerplate for my writing. |
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.
Pull Request Overview
This PR adds comprehensive documentation for configuring Microsoft Entra ID authentication with Coder using OpenID Connect (OIDC). The documentation provides step-by-step instructions for setting up authentication integration between Coder and Microsoft's identity platform.
- Creates a new Microsoft Entra ID OIDC authentication guide
- Provides detailed Azure app registration setup instructions
- Includes Coder configuration examples with environment variables
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
You would also want to add this to the docs/manifest.json as well You can use the commit from when we made the google doc for reference. |
Thanks! Done! |
|
I added additional information regarding changing your access URL, intended to make this another PR, but whoops got bundled in. Updating desc on this PR to reflect those changes. |
|
Adjusted changes as requested by @david-fraley pr-title checks are a pain, just ignore, I think the title isn't the issue at hand. |
Updated the PR title for ya to match the regex. You needed the |
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.
Nice work; thank you for helping us with our docs @PhoenixSheppy!
|
Thanks for the contribution @PhoenixSheppy ❤️ |
❤️ No problem guys, thanks for creating such wonderful software! |
|
Could I get some eyes on this, I'm not sure why, but this has been open for a while now and hasn't been merged after approval. Thank you. |
Propose Microsoft Entra ID OIDC Directions for Admin Documentation based on my personal experience / setup.
Propose information on changing access URL in Tutorials -> FAQs