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 env config / WithClientCert doesn't reload certificates #4683

Open
abh opened this issue Oct 31, 2023 · 1 comment
Open

OTLP env config / WithClientCert doesn't reload certificates #4683

abh opened this issue Oct 31, 2023 · 1 comment
Labels
blocked:specification Waiting on clarification of the OpenTelemetry specification before progress can be made enhancement New feature or request

Comments

@abh
Copy link

abh commented Oct 31, 2023

Description

Certificates configured with OTEL_EXPORTER_OTLP_CLIENT_KEY and OTEL_EXPORTER_OTLP_CLIENT_CERTIFICATE aren't reloaded, so if they are short lived certificates they only work for a short time.

See also #3363

Environment

  • Go Version: 1.21.3
  • opentelemetry-go version: v1.19.0/v0.42.0/v0.0.7

Steps To Reproduce

  1. Use the configuration options OTEL_EXPORTER_OTLP_CLIENT_KEY and OTEL_EXPORTER_OTLP_CLIENT_CERTIFICATE.
  2. Run the process
  3. Wait for certificates to be replaced
  4. See error when the original certificates expire

Expected behavior

Certificates are reloaded, or have an option to be reloaded, when they change on disk.

Related issues / solutions in the collector:
open-telemetry/opentelemetry-collector#4737
open-telemetry/opentelemetry-collector#7619

@abh abh added the bug Something isn't working label Oct 31, 2023
@pellared pellared added enhancement New feature or request and removed bug Something isn't working labels Oct 31, 2023
@pellared
Copy link
Member

  1. Is there OTLP exporter in any other language who works that way?
  2. This behavior is not specified in https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/protocol/exporter.md.

have an option to be reloaded, when they change on disk.

I think you would have to write custom code to handle it.

@pellared pellared added the blocked:specification Waiting on clarification of the OpenTelemetry specification before progress can be made label Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked:specification Waiting on clarification of the OpenTelemetry specification before progress can be made enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants