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

-Dotel.instrumentation.http.server.capture-request-headers=Cookie may cause http request 404 #11464

Closed
pepeshore opened this issue May 27, 2024 · 0 comments · Fixed by #11469
Closed
Labels
bug Something isn't working needs triage New issue that requires triage

Comments

@pepeshore
Copy link

Describe the bug

when I run my demo with
java -javaagent:opentelemetry-javaagent.jar -Dotel.instrumentation.http.server.capture-request-headers=Cookie spring-tomcat-jdk17-0.0.1-SNAPSHOT.jar
All HTTP requests return a 404 error
And I notice that if the request has no header named Cookie, the request returns successful

Steps to reproduce

  1. Download my demo spring-tomcat-jdk17-0.0.1-SNAPSHOT.zip
  2. Change the file extension from .zip to .jar.
  3. Run the demo with command java -javaagent:opentelemetry-javaagent.jar -Dotel.instrumentation.http.server.capture-request-headers=Cookie spring-tomcat-jdk17-0.0.1-SNAPSHOT.jar
  4. Initiate an HTTP request with command curl 'http://127.0.0.1:8080/tomcat/hello' -H 'accept: text/html' , a string 'hello' is returned
  5. Initiate an HTTP request with command curl 'http://127.0.0.1:8080/tomcat/hello' -H 'accept: text/html' -H 'cookie: ca=hbcda', a 404 error occurs

Expected behavior

a string 'hello' is returned

Actual behavior

a 404 error occurs

Javaagent or library instrumentation version

2.4

Environment

JDK:
OS:

Additional context

No response

@pepeshore pepeshore added bug Something isn't working needs triage New issue that requires triage labels May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage New issue that requires triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant