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

4.x WebClient Security Digest Authentication #7207

Closed
Tracked by #7219
romain-grecourt opened this issue Jul 17, 2023 · 1 comment · Fixed by #7415
Closed
Tracked by #7219

4.x WebClient Security Digest Authentication #7207

romain-grecourt opened this issue Jul 17, 2023 · 1 comment · Fixed by #7415
Assignees
Labels
4.x Version 4.x Níma Helidon Níma security team-fix This issue is available for team members to be fixed. Should not require deep knowledge of Helidon webclient
Milestone

Comments

@romain-grecourt
Copy link
Contributor

  • Helidon Version: 4.0.0-ALPHA6

We need an equivalent to org.glassfish.jersey.client.authentication.HttpAuthenticationFeature

This should support passing authentication directly to the client in form of properties.

client.get(uri)
    .property(HTTP_AUTHENTICATION_USERNAME, username)
    .property(HTTP_AUTHENTICATION_PASSWORD, password)
    .request();
@romain-grecourt romain-grecourt added this to the 4.0.0-M2 milestone Jul 17, 2023
@romain-grecourt romain-grecourt changed the title 4.x WebClient Security Authentication 4.x WebClient Security Digest Authentication Jul 17, 2023
@m0mus m0mus added the team-fix This issue is available for team members to be fixed. Should not require deep knowledge of Helidon label Jul 20, 2023
@tvallin tvallin self-assigned this Jul 24, 2023
@tomas-langer
Copy link
Member

tomas-langer commented Aug 21, 2023

This is supported for BasicAuthentication and Jwt (both SE and MP). It cannot be supported for digest authentication, as we do not support outbound security with HTTP Digest.
I will refactor the constants used to have a single set on a shared place (currently duplicated in JwtAuthProvider and HttpBasicAuthProvider)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x Version 4.x Níma Helidon Níma security team-fix This issue is available for team members to be fixed. Should not require deep knowledge of Helidon webclient
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants