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

OTLP exporter defaults to insecure connection #1769

Closed
MikeGoldsmith opened this issue Nov 14, 2022 · 3 comments
Closed

OTLP exporter defaults to insecure connection #1769

MikeGoldsmith opened this issue Nov 14, 2022 · 3 comments
Assignees
Labels
bug Something isn't working do-not-stale

Comments

@MikeGoldsmith
Copy link
Member

MikeGoldsmith commented Nov 14, 2022

Describe your environment
When trying to configure the SDK to send telemetry to a backend, the default OTLP exporter behaviour is to send using an unsecured connection.

This is not intuitive behaviour and goes against the specification. The environment variables are different, but the intent is the *_INSECURE environment variables default to false, meaning a secure connection is used.

Two nice to have:

  • update the SDKs environment variable usage to match the spec
  • if no environment variable is set, determine whether a secure connection is required using the endpoint protocol (eg http:// = insecure, https:// = secure)

Steps to reproduce
Setup SDK to send telemetry to an endpoint that required a secure connection (eg https://).

What is the expected behavior?
Telemetry successfully be exported to the configured endpoint.

What is the actual behavior?
Telemetry is not exported with an invalid protocol error. To get it to work, the SDK required the OTEL_EXPORTER_OTLP_SSL_ENABLE set to true.

@MikeGoldsmith MikeGoldsmith added the bug Something isn't working label Nov 14, 2022
@lalitb
Copy link
Member

lalitb commented Nov 16, 2022

Also, need to change the env variable used from OTEL_EXPORTER_OTLP_TRACES_SSL_ENABLE to OTEL_EXPORTER_OTLP_INSECURE

@esigo esigo self-assigned this Nov 16, 2022
@github-actions
Copy link

This issue was marked as stale due to lack of activity.

@marcalff
Copy link
Member

marcalff commented Jan 8, 2024

@MikeGoldsmith

Thanks for the report.

I think this is addressed by:

Closing the issue as resolved.

Please reopen and comment if something is still not working properly.

@marcalff marcalff closed this as completed Jan 8, 2024
@marcalff marcalff assigned marcalff and unassigned esigo Jan 8, 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 do-not-stale
Projects
None yet
Development

No branches or pull requests

4 participants