Kludex opened a new pull request, #44346: URL: https://github.com/apache/airflow/pull/44346
I'm trying to connect to [Logfire](https://logfire.pydantic.dev/docs/) without the need for an OTel collector, and this is getting in my way. Without the `headers` being set, I can do: ```bash export OTEL_EXPORTER_OTLP_HEADERS='Authorization=...' ``` And I can connect to Logfire - ...but this also benefits any other OpenTelemetry backend. > [!IMPORTANT] > The `OTLPSpanExporter` sets the `Content-Type` to `application/x-protobuf` by default. This PR may break current users that depend on the `Content-Type` being checked. As an alternative, the `OTLPSpanExporter` receives a `requests.Session`, which there we can pass the `content-type`. I think the only way to not break current users but allow the OTel package to get the environment variables as expected is to create the `requests.Session`, but also create an `otel_content_type` setting with the default `application/json`. That said... I think it was just a mistake to add this header in the first place. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
