open-telemetry / opentelemetry-cpp-contrib

https://opentelemetry.io/
Apache License 2.0
121 stars 128 forks source link

NginxModuleRequestHeaders and NginxModuleResponseHeaders are case sensitives #458

Open fede843 opened 2 weeks ago

fede843 commented 2 weeks ago

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.