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

[WFCORE-4485] Support for multiple security realms - Distributed Identities #291

Merged
merged 1 commit into from Aug 12, 2020

Conversation

nekdozjam
Copy link
Contributor


=== Hard Requirements

Add DistributedRealm to Elytron, which will be a SecurityRealm wrapper containing a list of SecurityRealms allowing to sequentially invoke them until one succeeds.

Choose a reason for hiding this comment

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

What happens in case a realm is not available? Will the authentication fail in such case?


== Test Plan

* TBD

Choose a reason for hiding this comment

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

Can you update this section please?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated


== Requirements

=== Hard Requirements

Choose a reason for hiding this comment

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

Additionally this use case should consider how the authorization information could be loaded from multiple sources and merged.

How is it taken into account?

Copy link
Contributor Author

@nekdozjam nekdozjam May 8, 2020

Choose a reason for hiding this comment

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

For this use case we already have a specific resource - aggregate-realm

[ ] QE

=== Affected Projects or Components

Choose a reason for hiding this comment

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

What about WildFly (Documentation and Tests Only)?

@jamezp jamezp merged commit e659bb1 into wildfly:master Aug 12, 2020
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.

None yet

3 participants