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

Edit docs for server.oidc_authentication.autologin cluster setting #18384

Merged
merged 3 commits into from May 15, 2024

Conversation

dhartunian
Copy link
Contributor

This setting is not required to be true and can be set based on customer preference.

@dhartunian dhartunian requested a review from mikeCRL March 14, 2024 18:45
Copy link

github-actions bot commented Mar 14, 2024

Copy link

netlify bot commented Mar 14, 2024

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit 47ff0d6
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-api-docs/deploys/66442a1b94c4f000080f7f6f

Copy link

netlify bot commented Mar 14, 2024

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit 47ff0d6
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-interactivetutorials-docs/deploys/66442a1bf925f60008928a9d

Copy link

netlify bot commented Mar 14, 2024

Netlify Preview

Name Link
🔨 Latest commit 47ff0d6
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-docs/deploys/66442a1ba0404a0007195d86
😎 Deploy Preview https://deploy-preview-18384--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@florence-crl florence-crl left a comment

Choose a reason for hiding this comment

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

LGTM pending suggested formatting change

@@ -76,7 +76,7 @@ You must configure the [cluster settings]({% link {{ page.version.version }}/clu
| `server.oidc_authentication.scopes` | A space-delimited list of the [OAuth scopes](https://openid.net/specs/openid-connect-core-1_0.html#ScopeClaims) being requested for an Access Token. The `openid` and `email` scopes must be included.<br>**Example:** `openid profile email`.
| `server.oidc_authentication.claim_json_key` | The field/key used to identify the user from the external identity provider's [ID Token](https://openid.net/specs/openid-connect-core-1_0.html#IDToken).
| `server.oidc_authentication.principal_regex` | Regex used to map the external identity key to a SQL user. For example: `^([^@]+)@[^@]+$` matches any email address (defined as a string containing one `@` sign) and extracts a username from the string to the left of `@`, whereas `^(.+)$` maps the claim directly to a principal.
| `server.oidc_authentication.autologin` | Must be set to `true`.
| `server.oidc_authentication.autologin` | A Boolean that enables or disables automatic login with SSO when the DB Console is loaded. If set to false, the user will have to click "Login with OIDC" before they're authenticated.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| `server.oidc_authentication.autologin` | A Boolean that enables or disables automatic login with SSO when the DB Console is loaded. If set to false, the user will have to click "Login with OIDC" before they're authenticated.
| `server.oidc_authentication.autologin` | A Boolean that enables or disables automatic login with SSO when the DB Console is loaded. If set to `false` (the default), the user will have to click **Login with OIDC** before they're authenticated.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. Also added a setting which I realized was missing and ported changes to last 2 versions.

The `server.oidc_authentication.autologin` setting is not required to
be `true` and can be set based on customer preference.

Added docs for the `server.oidc_authentication.button_text` setting
which has existed for all supported versions.
@dhartunian dhartunian merged commit c5f949e into main May 15, 2024
6 checks passed
@dhartunian dhartunian deleted the dhartunian-patch-1 branch May 15, 2024 03:54
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.

None yet

2 participants