-
Notifications
You must be signed in to change notification settings - Fork 914
docs: add details on external authentication priority #17164
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
Merged
Merged
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
911c820
add git auth in workspaces section; explain external auth vs ssh auth…
EdwardAngert 5166509
Update docs/admin/external-auth.md
EdwardAngert 58bbd31
Merge branch 'main' into 16875-git-workspace-auth
EdwardAngert 368b131
docs: clarify that CODER_EXTERNAL_AUTH_0_ID is used in callback URLs
bpmct aaec21d
more consistent examples
EdwardAngert 64dacc7
Update docs/admin/external-auth.md
EdwardAngert da3c83c
Update docs/admin/external-auth.md
EdwardAngert 6ba6b94
more consistent example urls
EdwardAngert d684ca2
better examples
EdwardAngert 8d481de
feat: set icons for each type of notification (#17115)
BrunoQuaresma fffa8c0
feat: add app status tracking to the backend (#17163)
kylecarbs 435c6a1
feat(cli): add `coder exp mcp` command (#17066)
johnstcn 83e4bc9
chore: bump vite from 5.4.15 to 5.4.16 in /site (#17176)
dependabot[bot] 1c3e054
chore: update msw to 2.4.8 (#17167)
sreya 41f6009
chore: pin goimports to 0.31.0 (#17177)
sreya 35cfc21
chore: pin various dependencies in CI files (#17180)
sreya de29681
ci: check go versions are consistent (#17149)
johnstcn b8a8cd8
chore(mcp): fix test flakes (#17183)
johnstcn 68c21bd
chore: improve error logging in TestServer/EphemeralDeployment (#17184)
hugodutka c19be18
fix: remove shared mutable state between oidc tests (#17179)
ethanndickson 556d972
fix(mcp): report task status correctly (#17187)
johnstcn 8bcda22
fix(site): standardize headers for Admin Settings page (#16911)
Parkreiner 0bfaae8
docs: add new cursor and windsurf docs (#17092)
EdwardAngert 5508c56
fix(cli): exp mcp: remove unnecessary cli flag (#17190)
johnstcn 4788ff0
feat: add frontend for app statuses (#17178)
kylecarbs bfc852e
docs: update SMTP configuration in notifications docs (#17161)
EdwardAngert c58f378
fix: convert workspace id in db2sdk.WorkspaceAppStatus (#17201)
kylecarbs 5f1d359
feat(cli): implement exp mcp configure claude-code command (#17195)
johnstcn f86b434
feat: add the ability to hide preset parameters (#17168)
SasSwart 5b39e95
Merge branch 'main' into 16875-git-workspace-auth
EdwardAngert a039753
Merge branch 'main' into 16875-git-workspace-auth
EdwardAngert File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 feel like that this whole SSH authentication section should not be here.
It should probably go to admin/users.
I don't have a strong opinion so will leave this to you.
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.
you might be right. I found an SSH Keys section in secrets.md that starts to explain this, but it doesn't quite fit there either
I think this fits better as a user guide. I'll leave the section here for now as it is something that an admin can help their user with, and we should aim to include this in future user-facing docs