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

Update env-vars-settings document in admin-guide: support for variables related to static login #652

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

Conversation

ism-k
Copy link

@ism-k ism-k commented Apr 26, 2022

Background

Created in response to a comment in this thread:
getredash/contrib-helm-chart#122

Problem

The current list in the Administrator's Guide does not list any environment variables for SAML static login.
I find REDASH_SAML_SSO_URL to be particularly important as it is a required configuration item to enable SAML login in Redash v10.

Change

The user-guide has a related description, so I have tried to correspond with this one:

### Static SAML
Static configuration requires these fields:
- **SAML Single Sign-on URL** is the URL at your IDP where users will be redirected when they click the _SAML Login_ button in Redash.
- **SAML Entity ID** should be the URL to your Redash instance.
- **SAML x509 cert** will be provided by your IDP.

@ism-k ism-k marked this pull request as ready for review April 26, 2022 16:17
@ism-k ism-k force-pushed the fix/document_env-vars-settings branch from 2092800 to 3ec1cd1 Compare May 2, 2022 01:41
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.

1 participant