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.

Sign up for GitHub

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

Jump to bottom

Simplify configuration to use different OpenIdConfiguration per webapp. #7042

Closed
lachlan-roberts opened this issue Oct 25, 2021 · 1 comment
Closed

Simplify configuration to use different OpenIdConfiguration per webapp. #7042

lachlan-roberts opened this issue Oct 25, 2021 · 1 comment
Assignees
@lachlan-roberts
Labels
Enhancement

Comments

@lachlan-roberts
Copy link
Contributor

Currently to use different OpenID providers for different webapps you must configure everything in the context xml for each webapp and manually add security handler with Authenticator and OpenLoginService etc.

It would be a lot better if this could be simplified if this could be discovered and set up automatically based on realm name. Similar to how the LoginService is found by the SecurityHandler from a bean on the Server.
lachlan-roberts added a commit that referenced this issue Oct 25, 2021
@lachlan-roberts
Issue #7042 - Allow OpenIdConfiguration to be selected based on realm…
b8af571 
… name.

Signed-off-by: Lachlan Roberts <[email protected]>
@lachlan-roberts lachlan-roberts mentioned this issue Oct 25, 2021
Merged
@lachlan-roberts lachlan-roberts self-assigned this Oct 28, 2021
lachlan-roberts added a commit that referenced this issue Nov 4, 2021
@lachlan-roberts
Issue #7042 - add name arguments to jetty-openid.xml
deb4e20 
Signed-off-by: Lachlan Roberts <[email protected]>
lachlan-roberts added a commit that referenced this issue Nov 9, 2021
@lachlan-roberts
Issue #7042 - Changes from review
f12d6f5 
Signed-off-by: Lachlan Roberts <[email protected]>
lachlan-roberts added a commit that referenced this issue Nov 10, 2021
@lachlan-roberts
Issue #7042 - if only 1 OpenIdConfig available select it regardless o…
2a3c65c 
…f realm name

Signed-off-by: Lachlan Roberts <[email protected]>
lachlan-roberts added a commit that referenced this issue Nov 10, 2021
@lachlan-roberts
Issue #7042 - remove init param to set authenticateNewUsers per webapp
2a51972 
Signed-off-by: Lachlan Roberts <[email protected]>
lachlan-roberts added a commit that referenced this issue Nov 23, 2021
@lachlan-roberts
Merge pull request #7043 from eclipse/jetty-10.0.x-7042-OpenIdConfigu…
d755e3a 
…ration

Issue #7042 - Allow OpenIdConfiguration to be selected based on realm name
@lachlan-roberts
Copy link
Contributor Author

Merged #7043.

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

@lachlan-roberts lachlan-roberts

Labels
Enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lachlan-roberts