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

NginxModuleRequestHeaders and NginxModuleResponseHeaders are case sensitives #458

Open
fede843 opened this issue Jun 13, 2024 · 0 comments
Open

NginxModuleRequestHeaders and NginxModuleResponseHeaders are case sensitives #458

fede843 opened this issue Jun 13, 2024 · 0 comments
Labels
Webserver This represents the otel-webserver-module in the instrumentation directory

Comments

@fede843
Copy link

fede843 commented Jun 13, 2024

We are using since a while otel-webserver-module for NGINX.
Is there a good reason why NginxModuleRequestHeaders and NginxModuleResponseHeaders are case sensitives?
HTTP 1.1 and 2.0 states in their RFCs that they are insensitive.
It would be nice to get it aligned with that. Or maybe a new flag to not break current behaviour. In our case we need to double define headers here due to this issue. Some 3rd parties are sending camel case headers. It has become an issue.
@marcalff marcalff added the Webserver This represents the otel-webserver-module in the instrumentation directory label Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Webserver This represents the otel-webserver-module in the instrumentation directory
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fede843 @marcalff