Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on May 18, 2026. It is now read-only.

feat(oauth): add oidc username claim#357

Merged
stonith404 merged 2 commits into
stonith404:mainfrom
qingfua:feat/oidc-username-claim
Dec 20, 2023
Merged

feat(oauth): add oidc username claim#357
stonith404 merged 2 commits into
stonith404:mainfrom
qingfua:feat/oidc-username-claim

Conversation

@qingfua
Copy link
Copy Markdown
Contributor

@qingfua qingfua commented Dec 14, 2023

close #355

@qingfua qingfua marked this pull request as ready for review December 14, 2023 14:55
@stonith404
Copy link
Copy Markdown
Owner

Thanks :) I'm currently not at home but I'll review it at the start of the next week.

Comment thread backend/src/oauth/provider/genericOidc.provider.ts Outdated
@qingfua
Copy link
Copy Markdown
Contributor Author

qingfua commented Dec 19, 2023

@stonith404 Anything else I should update?

Copy link
Copy Markdown
Owner

@stonith404 stonith404 left a comment

Choose a reason for hiding this comment

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

Sorry, I was interrupted while reviewing. Yes this looks fine, thanks!

@stonith404
Copy link
Copy Markdown
Owner

@zz5840 Can we merge?

@qingfua
Copy link
Copy Markdown
Contributor Author

qingfua commented Dec 20, 2023

@stonith404 Yeah, please merge it.

@stonith404 stonith404 merged commit 3ea52a2 into stonith404:main Dec 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

❓ Question: Internal Error while setting the OIDC

2 participants