Skip to content

[Docs] Add IdP-initiated SSO instructions to Grafana SAML guide #28058

Merged
rudream merged 1 commit intomasterfrom
yassine/docs/grafana-saml
Jul 4, 2023
Merged

[Docs] Add IdP-initiated SSO instructions to Grafana SAML guide #28058
rudream merged 1 commit intomasterfrom
yassine/docs/grafana-saml

Conversation

@rudream
Copy link
Copy Markdown
Contributor

@rudream rudream commented Jun 20, 2023

Purpose

Adds documentation for getting SAML IdP-intiated SSO working with Grafana.

Our current guide for setting up Teleport as SAML IdP for Grafana is missing instructions for how to get IdP-initiated SSO working. Particularly, the fact that the relay_state in grafana.ini has to be set to "" for it to work (since it's currently hardcoded to be blank on the Teleport side). This currently isn't documented anywhere in our docs and attempting to log into Grafana via IdP-initiated SSO without it will fail with the following error:

image

Demo

image

@alexfornuto alexfornuto self-assigned this Jun 21, 2023
Copy link
Copy Markdown
Contributor

@alexfornuto alexfornuto left a comment

Choose a reason for hiding this comment

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

I'm unable to verify as my trial license for Grafana enterprise ran out after writing this doc, so.... LGTM?

@rudream
Copy link
Copy Markdown
Contributor Author

rudream commented Jul 4, 2023

I'm unable to verify as my trial license for Grafana enterprise ran out after writing this doc, so.... LGTM?

@alexfornuto This worked for my local setup running Grafana enterprise v9.5.2.

@rudream rudream added this pull request to the merge queue Jul 4, 2023
Merged via the queue into master with commit b2cd70c Jul 4, 2023
@rudream rudream deleted the yassine/docs/grafana-saml branch July 4, 2023 16:01
@public-teleport-github-review-bot
Copy link
Copy Markdown

@rudream See the table below for backport results.

Branch Result
branch/v12 Create PR
branch/v13 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants