Add origin configuration to authc providers#239993
Merged
rgodfrey-elastic merged 14 commits intoelastic:mainfrom Nov 6, 2025
Merged
Add origin configuration to authc providers#239993rgodfrey-elastic merged 14 commits intoelastic:mainfrom
rgodfrey-elastic merged 14 commits intoelastic:mainfrom
Conversation
Contributor
🔍 Preview links for changed docs |
…filter available providers based on the origin configuration and the current browser window origin. Also filtered available providers based on the origin header and the configured provider origin properties # Conflicts: # x-pack/platform/plugins/shared/security/server/authentication/authenticator.ts
…ge message to use translate function
37fa181 to
34bb5d3
Compare
….com:rgodfrey-elastic/kibana into feature/add-origin-config-to-authc-providers
Contributor
|
Pinging @elastic/kibana-security (Team:Security) |
Co-authored-by: florent-leborgne <florent.leborgne@elastic.co>
Contributor
|
ACK: will review later today or tomorrow, sorry for the delay! |
azasypkin
reviewed
Oct 31, 2025
Contributor
azasypkin
left a comment
There was a problem hiding this comment.
Great job! I've left just a few questions and nits.
...orm/plugins/shared/security/public/authentication/login/components/login_form/login_form.tsx
Outdated
Show resolved
Hide resolved
...lugins/shared/security/public/authentication/login/components/login_form/login_form.test.tsx
Outdated
Show resolved
Hide resolved
...lugins/shared/security/public/authentication/login/components/login_form/login_form.test.tsx
Outdated
Show resolved
Hide resolved
x-pack/platform/plugins/shared/security/server/authentication/providers/base.ts
Outdated
Show resolved
Hide resolved
x-pack/platform/plugins/shared/security/server/authentication/authenticator.ts
Outdated
Show resolved
Hide resolved
rgodfrey-elastic
commented
Nov 4, 2025
...lugins/shared/security/public/authentication/login/components/login_form/login_form.test.tsx
Outdated
Show resolved
Hide resolved
Contributor
Author
|
@elasticmachine merge upstream |
azasypkin
approved these changes
Nov 5, 2025
Contributor
azasypkin
left a comment
There was a problem hiding this comment.
LGTM, tested locally - everything works as expected, thanks! Would you also mind updating the Release Notes section in the issue description to better align with your updates in the doc (these are being used to compile release notes for every release)?
...lugins/shared/security/public/authentication/login/components/login_form/login_form.test.tsx
Outdated
Show resolved
Hide resolved
...lugins/shared/security/public/authentication/login/components/login_form/login_form.test.tsx
Outdated
Show resolved
Hide resolved
Contributor
Author
|
@elasticmachine merge upstream |
Contributor
💚 Build Succeeded
Metrics [docs]Async chunks
History
|
florent-leborgne
approved these changes
Nov 6, 2025
Contributor
florent-leborgne
left a comment
There was a problem hiding this comment.
Docs LGTM, thanks!
wildemat
pushed a commit
to wildemat/kibana
that referenced
this pull request
Nov 6, 2025
Closes [109525](elastic#109525) ## Summary - Added origin configuration to authc providers. - Changed login form to hide providers based on the origin configuration and the current browser window origin. - Filtered providers on the back end based on the origin header and the configured provider origin properties. - Origin configuration is optional and can be one value or an array of values. - All values provided in the origin config must be a valid URI - An error is displayed in the UI if there are no valid auth providers for the domain ### Example 1 ``` xpack.security.authc.providers: basic.basic1: order: 0 origin: [http://127.0.0.1:5601, http://localhost:5601, https://elastic.com] saml.saml1: order: 1 realm: saml1 origin: http://127.0.0.1:5601 saml.saml2: order: 2 realm: saml2 origin: http://localhost:5601 saml.saml3: order: 3 realm: saml3 origin: [http://127.0.0.1:5601, http://localhost:5601, https://elastic.com] saml.saml4: order: 4 realm: saml4 ``` <img width="735" height="585" alt="image" src="https://github.com/user-attachments/assets/d691f692-6470-4d59-aba1-bc598b4b49a2" /> <img width="725" height="597" alt="image" src="https://github.com/user-attachments/assets/28a61462-ef00-484f-b2c9-1816bc50fc54" /> ### Example 2 ``` xpack.security.authc.providers: basic.basic1: order: 0 origin: [http://127.0.0.1:5601, https://elastic.com] saml.saml1: order: 1 realm: saml1 origin: https://elastic.com ``` <img width="772" height="443" alt="image" src="https://github.com/user-attachments/assets/9c332a42-2a48-43ea-b4c5-0d9ab6660b6a" /> ## Release Notes Adds the ability to specify the origin(s) where an authentication provider will appear to users in the Login Selector UI. --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: florent-leborgne <florent.leborgne@elastic.co> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
rgodfrey-elastic
added a commit
to elastic/docs-content
that referenced
this pull request
Nov 6, 2025
## Summary Added note about new `origin` setting available for authentication providers added with elastic/kibana#239993
eokoneyo
pushed a commit
to eokoneyo/kibana
that referenced
this pull request
Dec 2, 2025
Closes [109525](elastic#109525) ## Summary - Added origin configuration to authc providers. - Changed login form to hide providers based on the origin configuration and the current browser window origin. - Filtered providers on the back end based on the origin header and the configured provider origin properties. - Origin configuration is optional and can be one value or an array of values. - All values provided in the origin config must be a valid URI - An error is displayed in the UI if there are no valid auth providers for the domain ### Example 1 ``` xpack.security.authc.providers: basic.basic1: order: 0 origin: [http://127.0.0.1:5601, http://localhost:5601, https://elastic.com] saml.saml1: order: 1 realm: saml1 origin: http://127.0.0.1:5601 saml.saml2: order: 2 realm: saml2 origin: http://localhost:5601 saml.saml3: order: 3 realm: saml3 origin: [http://127.0.0.1:5601, http://localhost:5601, https://elastic.com] saml.saml4: order: 4 realm: saml4 ``` <img width="735" height="585" alt="image" src="https://github.com/user-attachments/assets/d691f692-6470-4d59-aba1-bc598b4b49a2" /> <img width="725" height="597" alt="image" src="https://github.com/user-attachments/assets/28a61462-ef00-484f-b2c9-1816bc50fc54" /> ### Example 2 ``` xpack.security.authc.providers: basic.basic1: order: 0 origin: [http://127.0.0.1:5601, https://elastic.com] saml.saml1: order: 1 realm: saml1 origin: https://elastic.com ``` <img width="772" height="443" alt="image" src="https://github.com/user-attachments/assets/9c332a42-2a48-43ea-b4c5-0d9ab6660b6a" /> ## Release Notes Adds the ability to specify the origin(s) where an authentication provider will appear to users in the Login Selector UI. --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: florent-leborgne <florent.leborgne@elastic.co> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes 109525
Summary
Example 1
Example 2
Release Notes
Adds the ability to specify the origin(s) where an authentication provider will appear to users in the Login Selector UI.