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 <h.dost@criteo.com>
open-telemetry · Feb 18, 2024 · 8aee214 · 8aee214
1 parent 1be83bb
commit 8aee214
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/opentelemetry-otlp/CHANGELOG.md b/opentelemetry-otlp/CHANGELOG.md
@@ -2,10 +2,15 @@
 
 ## vNext
 
-- **Breaking** Remove support for surf HTTP client [#1537](https://github.com/open-telemetry/opentelemetry-rust/pull/1537)
+### Changed
 - Update to tonic 0.11 and prost 0.12 (#1536)
+- Fix `tonic()` to the use correct port. [#1556](https://github.com/open-telemetry/opentelemetry-rust/pull/1556)
+
+### 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
 
 ### Added

diff --git 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,