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

Dynamic discovery provider locator can't resolved principal #5988

Conversation

lgwozniak
Copy link
Contributor

Hi,

We use Dynamic Discovery Provider, to determine Client to authentication ( We have Office365 and soon OpenAM). We have about 150 auth request per minute. When I try upgrade library from 6.5 to 7.0 there is a problem to determine where forward client.

I found that in class DefaultDelegatedAuthenticationDynamicDiscoveryProviderLocator was added Principal Resolver. Default Principal Resolver is EchoingPrincipalResolver called from ChainingPrincipalResolver, where EchoingPrincipalResolver always return NullPrincipal.

Result: It cannot find Client to redirect.

@mmoayyed
Copy link
Member

mmoayyed commented Feb 8, 2024

Thank you. Have you already run the style guide on this change?

@lgwozniak
Copy link
Contributor Author

I think i fix it.

apereocas-bot pushed a commit that referenced this pull request Feb 8, 2024
@mmoayyed
Copy link
Member

mmoayyed commented Feb 9, 2024

merged via 2e142fe799a81e5ea890d1b17350a327d19c047a

@mmoayyed mmoayyed closed this Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants