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

[Docs] Add auth_provider_hint to authentication docs #104132

Merged
merged 2 commits into from
Jul 6, 2021

Conversation

jportner
Copy link
Contributor

@jportner jportner commented Jul 1, 2021

Some users want to show the login selector, but also to craft specific URLs for individual authentication providers. This is actually as of 7.11+ with the auth_provider_hint URL query parameter.

This PR adds a tip to the Kibana Authentication docs to mention that parameter.

Docs preview: https://kibana_104132.docs-preview.app.elstc.co/diff

@jportner jportner requested a review from azasypkin July 1, 2021 15:59
@jportner jportner added the auto-backport Deprecated - use backport:version if exact versions are needed label Jul 1, 2021
@jportner jportner requested a review from a team July 1, 2021 16:01
Copy link
Contributor

@gchaps gchaps left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@azasypkin azasypkin left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

I was always hesitating to document this option to keep the right to change it at any time. But it seems we recommend it to our users in various corner cases more and more now that makes it a de facto public API we have to support anyway, so 👍 .

@@ -65,6 +65,10 @@ image::user/security/images/kibana-login.png["Login Selector UI"]

For more information, refer to <<authentication-security-settings, authentication security settings>>.

TIP: If you have multiple authentication providers configured, you can use the `auth_provider_hint` URL query parameter to create a deep
link to any provider and bypass the Login Selector UI. Using the `kibana.yml` above as an example, you can add `?auth_provider_hint=basic1`
Copy link
Member

Choose a reason for hiding this comment

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

optional nit: we may want to also add that it would work even if this particular option isn't displayed in Login Selector UI.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, it's good to know that. I think in the interest of brevity I will leave it as-is, but if we get questions about that down the road we can always add it 👍

@jportner jportner merged commit 57fdadb into master Jul 6, 2021
@jportner jportner deleted the auth_provider_hint_docs branch July 6, 2021 14:02
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 6, 2021
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 6, 2021
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 6, 2021
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 6, 2021
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 6, 2021
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.11
7.12
7.13
7.14
7.x

The backport PRs will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Jul 6, 2021
Co-authored-by: Joe Portner <5295965+jportner@users.noreply.github.com>
kibanamachine added a commit that referenced this pull request Jul 6, 2021
Co-authored-by: Joe Portner <5295965+jportner@users.noreply.github.com>
kibanamachine added a commit that referenced this pull request Jul 6, 2021
Co-authored-by: Joe Portner <5295965+jportner@users.noreply.github.com>
kibanamachine added a commit that referenced this pull request Jul 6, 2021
Co-authored-by: Joe Portner <5295965+jportner@users.noreply.github.com>
kibanamachine added a commit that referenced this pull request Jul 6, 2021
Co-authored-by: Joe Portner <5295965+jportner@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed docs release_note:skip Skip the PR/issue when compiling release notes v7.11.0 v7.12.0 v7.13.0 v7.14.0 v7.15.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants