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

HttpAttributes.SERVICE_ID not set when using jdk http client #11041

Closed
magnayn-mns opened this issue Jul 31, 2024 · 1 comment
Closed

HttpAttributes.SERVICE_ID not set when using jdk http client #11041

magnayn-mns opened this issue Jul 31, 2024 · 1 comment
Labels
info: good first issue Good for newcomers type: bug Something isn't working

Comments

@magnayn-mns
Copy link

Expected Behavior

Service ID should be propagated in attributes

Actual Behaviour

nothing is there - meaning that the OAuth2 token flow is not invoked.

Steps To Reproduce

Have a service defined with @client using OAuth2, and attempt to match with service-regex

Environment Information

No response

Example Application

No response

Version

4.2.3
@graemerocher graemerocher added type: bug Something isn't working info: good first issue Good for newcomers labels Aug 6, 2024
altro3 added a commit to altro3/micronaut-core that referenced this issue Nov 5, 2024
@altro3
Add lost SERVICE_ID request attribute with jdk http client.
37bcc14 
Fixed micronaut-projects#11041
@altro3 altro3 mentioned this issue Nov 5, 2024
Merged
yawkat pushed a commit that referenced this issue Nov 8, 2024
@altro3
Add lost SERVICE_ID request attribute with jdk http client. (#11309)
e4416f3 
Fixed #11041
@altro3
Copy link
Contributor

altro3 commented Nov 8, 2024

@graemerocher @yawkat You already merged fix. Issue can be closed

@yawkat yawkat closed this as completed Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
info: good first issue Good for newcomers type: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@graemerocher @yawkat @altro3 @magnayn-mns