From b714b28360d9b41978c577f9e75699e0967c371f Mon Sep 17 00:00:00 2001 From: Harold Dost Date: Sun, 18 Feb 2024 08:38:14 +0100 Subject: [PATCH] opentelemetry-otlp: Default the port correctly. Current behavior uses http-proto port even when there's no desire to use it. Relates #1509 Signed-off-by: Harold Dost --- opentelemetry-otlp/CHANGELOG.md | 8 +++++++- opentelemetry-otlp/src/exporter/tonic/mod.rs | 1 + 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/opentelemetry-otlp/CHANGELOG.md b/opentelemetry-otlp/CHANGELOG.md index 5a79ab9d23..61208b45c4 100644 --- a/opentelemetry-otlp/CHANGELOG.md +++ b/opentelemetry-otlp/CHANGELOG.md @@ -2,8 +2,14 @@ ## vNext -- **Breaking** Remove support for surf HTTP client [#1537](https://github.com/open-telemetry/opentelemetry-rust/pull/1537) +### Fixed +- Fix `tonic()` to the use correct port. [#1556](https://github.com/open-telemetry/opentelemetry-rust/pull/1556) + +### Changed - Update to tonic 0.11 and prost 0.12 (#1536) + +### Removed +- **Breaking** Remove support for surf HTTP client [#1537](https://github.com/open-telemetry/opentelemetry-rust/pull/1537) - Remove support for grpcio transport (#1534) ## v0.14.0 diff --git a/opentelemetry-otlp/src/exporter/tonic/mod.rs b/opentelemetry-otlp/src/exporter/tonic/mod.rs index 7f5f8709b2..373154bf28 100644 --- a/opentelemetry-otlp/src/exporter/tonic/mod.rs +++ b/opentelemetry-otlp/src/exporter/tonic/mod.rs @@ -149,6 +149,7 @@ impl Default for TonicExporterBuilder { TonicExporterBuilder { exporter_config: ExportConfig { protocol: crate::Protocol::Grpc, + endpoint: default_endpoint(crate::Protocol::Grpc), ..Default::default() }, tonic_config,