temporalio / sdk-core

Core Temporal SDK that can be used as a base for language specific Temporal SDKs
MIT License
266 stars 70 forks source link

[Bug] OpenTelemetry export fails over TLS #801

Closed cretz closed 3 weeks ago

cretz commented 3 weeks ago

Describe the bug

Newer Tonic doesn't default to using the system root CA list anymore. We fixed this for client connections at #785, but we were unaware that this applies to OTel too. Basically we need to fix our TonicExporterBuilder options the same way users did at https://github.com/open-telemetry/opentelemetry-rust/issues/2008#issuecomment-2292294188.