Update default OTLP port (#388)

Author: drexler

opentelemetry-otlp/README.md

@@ -38,7 +38,7 @@ First make sure you have a running version of the opentelemetry collector you
 want to send data to:
 
 ```shell
-$ docker run -p 55680:55680 otel/opentelemetry-collector-dev:latest
+$ docker run -p 4317:4317 otel/opentelemetry-collector-dev:latest
 ```
 
 Then install a new pipeline with the recommended defaults to start exporting
@@ -116,7 +116,7 @@ fn main() -> Result<(), Box<dyn std::error::Error + Send + Sync + 'static>> {
     map.insert_bin("trace-proto-bin", MetadataValue::from_bytes(b"[binary data]"));
 
     let (tracer, _uninstall) = opentelemetry_otlp::new_pipeline()
-        .with_endpoint("localhost:55680")
+        .with_endpoint("localhost:4317")
         .with_protocol(Protocol::Grpc)
         .with_metadata(map)
         .with_timeout(Duration::from_secs(3))

opentelemetry-otlp/src/lib.rs

@@ -12,7 +12,7 @@
 //! you want to send data to:
 //!
 //! ```shell
-//! $ docker run -p 55680:55680 otel/opentelemetry-collector-dev:latest
+//! $ docker run -p 4317:4317 otel/opentelemetry-collector-dev:latest
 //! ```
 //!
 //! Then install a new pipeline with the recommended defaults to start exporting
@@ -83,7 +83,7 @@
 //!     map.insert_bin("trace-proto-bin", MetadataValue::from_bytes(b"[binary data]"));
 //!
 //!     let (tracer, _uninstall) = opentelemetry_otlp::new_pipeline()
-//!         .with_endpoint("localhost:55680")
+//!         .with_endpoint("localhost:4317")
 //!         .with_protocol(Protocol::Grpc)
 //!         .with_metadata(map)
 //!         .with_timeout(Duration::from_secs(3))

opentelemetry-otlp/src/span.rs

@@ -143,11 +143,13 @@ impl Into<grpcio::CompressionAlgorithms> for Compression {
     }
 }
 
+const DEFAULT_OTLP_PORT: u16 = 4317;
+
 impl Default for ExporterConfig {
     #[cfg(feature = "tonic")]
     fn default() -> Self {
         ExporterConfig {
-            endpoint: String::from("localhost:55680"),
+            endpoint: format!("localhost:{}", DEFAULT_OTLP_PORT),
             protocol: Protocol::Grpc,
             #[cfg(all(feature = "tonic", feature = "tls"))]
             tls_config: None,
@@ -161,7 +163,7 @@ impl Default for ExporterConfig {
     #[cfg(all(feature = "grpc-sys", not(feature = "tonic")))]
     fn default() -> Self {
         ExporterConfig {
-            endpoint: String::from("localhost:55680"),
+            endpoint: format!("localhost:{}", DEFAULT_OTLP_PORT),
             protocol: Protocol::Grpc,
             credentials: None,
             headers: None,