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.

Sign up for GitHub

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 oidc-Identity-provider.md #343

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

Update oidc-Identity-provider.md #343

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions sig-multitenancy/auth/oidc-Identity-provider.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ data:
clientID: '********'
clientSecret: '********'
issuer: https://accounts.google.com
redirectURL: 'http://ks-console/oauth/redirect'
redirectURL: 'http://ks-console/oauth/redirect/google'
Copy link
Member

Choose a reason for hiding this comment

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

It would be great if you could provide some references here. For example, document, code line. Because I don't familiar with the feature. I cannot determine if it's correct.

kind: ConfigMap
name: kubesphere-config
namespace: kubesphere-system
Expand All @@ -43,4 +43,4 @@ For the above example:
| preferredUsernameKey | Configurable key which contains the preferred username claims. |
| emailKey | Configurable key which contains the email claims. |
| getUserInfo | GetUserInfo uses the userinfo endpoint to get additional claims for the token. This is especially useful where upstreams return "thin" id tokens. |
| insecureSkipVerify | Used to turn off TLS certificate verify. |
| insecureSkipVerify | Used to turn off TLS certificate verify. |