-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
OTLP exporters have improper default endpoint #4147
Comments
To begin with, we need to consider the implications of changing the OTLP exporter. Such a modification would fundamentally alter the default behavior, possibly causing disruptions in the existing setup. One significant concern is that changing the default would break most code that is using secure HTTPS endpoints. Presently, the Introducing a new factory functions with different defaults, that would be compliant with the specification, might introduce unnecessary confusion and expand the complexity of the API. Therefore, maintaining the current approach appears to be the most viable option. Instead, we might consider clearly documenting the default behavior and acknowledging our deviation from the specification in this aspect.
|
I plan to close it by end of this week. CC @open-telemetry/go-approvers |
SIG meeting: |
I created #4688 for documentation cleanup. |
Got
The OTLP default endpoints default to
https
(secure) scheme.Want
The OTLP default endpoints default to
http
(insecure) scheme.Per: https://github.com/open-telemetry/opentelemetry-specification/blob/v1.21.0/specification/protocol/exporter.md
The text was updated successfully, but these errors were encountered: