added build_default to MetricExporterBuilder

Author: tomDev5

opentelemetry-otlp/src/exporter/mod.rs

@@ -46,9 +46,14 @@ pub const OTEL_EXPORTER_OTLP_PROTOCOL_DEFAULT: &str = OTEL_EXPORTER_OTLP_PROTOCO
 /// Default protocol if no features are enabled.
 pub const OTEL_EXPORTER_OTLP_PROTOCOL_DEFAULT: &str = "";
 
-const OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_PROTOBUF: &str = "http/protobuf";
-const OTEL_EXPORTER_OTLP_PROTOCOL_GRPC: &str = "grpc";
-const OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_JSON: &str = "http/json";
+/// http-protobuf Exporter protocol
+pub const OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_PROTOBUF: &str = "http/protobuf";
+
+/// grpc Exporter protocol
+pub const OTEL_EXPORTER_OTLP_PROTOCOL_GRPC: &str = "grpc";
+
+/// http-json Exporter protocol
+pub const OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_JSON: &str = "http/json";
 
 /// Max waiting time for the backend to process each signal batch, defaults to 10 seconds.
 pub const OTEL_EXPORTER_OTLP_TIMEOUT: &str = "OTEL_EXPORTER_OTLP_TIMEOUT";

opentelemetry-otlp/src/lib.rs

@@ -401,8 +401,9 @@ pub use crate::exporter::tonic::{HasTonicConfig, WithTonicConfig};
 pub use crate::exporter::{
     HasExportConfig, WithExportConfig, OTEL_EXPORTER_OTLP_COMPRESSION, OTEL_EXPORTER_OTLP_ENDPOINT,
     OTEL_EXPORTER_OTLP_ENDPOINT_DEFAULT, OTEL_EXPORTER_OTLP_HEADERS, OTEL_EXPORTER_OTLP_PROTOCOL,
-    OTEL_EXPORTER_OTLP_PROTOCOL_DEFAULT, OTEL_EXPORTER_OTLP_TIMEOUT,
-    OTEL_EXPORTER_OTLP_TIMEOUT_DEFAULT,
+    OTEL_EXPORTER_OTLP_PROTOCOL_DEFAULT, OTEL_EXPORTER_OTLP_PROTOCOL_GRPC,
+    OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_JSON, OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_PROTOBUF,
+    OTEL_EXPORTER_OTLP_TIMEOUT, OTEL_EXPORTER_OTLP_TIMEOUT_DEFAULT,
 };
 
 /// Type to indicate the builder does not have a client set.

opentelemetry-otlp/src/logs.rs

@@ -8,10 +8,21 @@ use opentelemetry_sdk::{error::OTelSdkResult, logs::LogBatch};
 use std::fmt::Debug;
 use std::time;
 
-use crate::{ExporterBuildError, HasExportConfig, NoExporterBuilderSet};
+use crate::{
+    ExporterBuildError, HasExportConfig, NoExporterBuilderSet, OTEL_EXPORTER_OTLP_PROTOCOL,
+    OTEL_EXPORTER_OTLP_PROTOCOL_DEFAULT,
+};
 
 #[cfg(feature = "grpc-tonic")]
-use crate::{HasTonicConfig, TonicExporterBuilder, TonicExporterBuilderSet};
+use crate::{
+    HasTonicConfig, TonicExporterBuilder, TonicExporterBuilderSet, OTEL_EXPORTER_OTLP_PROTOCOL_GRPC,
+};
+
+#[cfg(feature = "http-proto")]
+use crate::OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_PROTOBUF;
+
+#[cfg(feature = "http-json")]
+use crate::OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_JSON;
 
 #[cfg(any(feature = "http-proto", feature = "http-json"))]
 use crate::{HasHttpConfig, HttpExporterBuilder, HttpExporterBuilderSet};
@@ -123,6 +134,24 @@ impl LogExporter {
         LogExporterBuilder::default()
     }
 
+    /// Builds a default [LogExporter] based on the value of `OTEL_EXPORTER_OTLP_PROTOCOL`.
+    pub fn build_default(self) -> Result<LogExporter, ExporterBuildError> {
+        match std::env::var(OTEL_EXPORTER_OTLP_PROTOCOL)
+            .unwrap_or(OTEL_EXPORTER_OTLP_PROTOCOL_DEFAULT.to_string())
+            .as_str()
+        {
+            #[cfg(feature = "grpc-tonic")]
+            OTEL_EXPORTER_OTLP_PROTOCOL_GRPC => Self::builder().with_tonic().build(),
+            #[cfg(feature = "http-proto")]
+            OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_PROTOBUF => Self::builder().with_http().build(),
+            #[cfg(feature = "http-json")]
+            OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_JSON => Self::builder().with_http().build(),
+            other => Err(ExporterBuildError::InternalFailure(format!(
+                "Invalid {OTEL_EXPORTER_OTLP_PROTOCOL}: {other}"
+            ))),
+        }
+    }
+
     #[cfg(any(feature = "http-proto", feature = "http-json"))]
     pub(crate) fn from_http(client: crate::exporter::http::OtlpHttpClient) -> Self {
         LogExporter {

opentelemetry-otlp/src/metric.rs

@@ -9,10 +9,22 @@ use crate::HasExportConfig;
 #[cfg(any(feature = "http-proto", feature = "http-json"))]
 use crate::{exporter::http::HttpExporterBuilder, HasHttpConfig, HttpExporterBuilderSet};
 
+#[cfg(feature = "http-proto")]
+use crate::exporter::OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_PROTOBUF;
+
+#[cfg(feature = "http-json")]
+use crate::exporter::OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_JSON;
+
 #[cfg(feature = "grpc-tonic")]
-use crate::{exporter::tonic::TonicExporterBuilder, HasTonicConfig, TonicExporterBuilderSet};
+use crate::{
+    exporter::{tonic::TonicExporterBuilder, OTEL_EXPORTER_OTLP_PROTOCOL_GRPC},
+    HasTonicConfig, TonicExporterBuilderSet,
+};
 
-use crate::{ExporterBuildError, NoExporterBuilderSet};
+use crate::{
+    ExporterBuildError, NoExporterBuilderSet, OTEL_EXPORTER_OTLP_PROTOCOL,
+    OTEL_EXPORTER_OTLP_PROTOCOL_DEFAULT,
+};
 
 use core::fmt;
 use opentelemetry_sdk::error::OTelSdkResult;
@@ -187,6 +199,24 @@ impl MetricExporter {
         MetricExporterBuilder::default()
     }
 
+    /// Builds a default [MetricExporter] based on the value of `OTEL_EXPORTER_OTLP_PROTOCOL`.
+    pub fn build_default(self) -> Result<MetricExporter, ExporterBuildError> {
+        match std::env::var(OTEL_EXPORTER_OTLP_PROTOCOL)
+            .unwrap_or(OTEL_EXPORTER_OTLP_PROTOCOL_DEFAULT.to_string())
+            .as_str()
+        {
+            #[cfg(feature = "grpc-tonic")]
+            OTEL_EXPORTER_OTLP_PROTOCOL_GRPC => Self::builder().with_tonic().build(),
+            #[cfg(feature = "http-proto")]
+            OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_PROTOBUF => Self::builder().with_http().build(),
+            #[cfg(feature = "http-json")]
+            OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_JSON => Self::builder().with_http().build(),
+            other => Err(ExporterBuildError::InternalFailure(format!(
+                "Invalid {OTEL_EXPORTER_OTLP_PROTOCOL}: {other}"
+            ))),
+        }
+    }
+
     #[cfg(feature = "grpc-tonic")]
     pub(crate) fn from_tonic(
         client: crate::exporter::tonic::metrics::TonicMetricsClient,

opentelemetry-otlp/src/span.rs

@@ -7,12 +7,18 @@ use std::fmt::Debug;
 use opentelemetry_sdk::error::OTelSdkResult;
 use opentelemetry_sdk::trace::SpanData;
 
-use crate::ExporterBuildError;
 #[cfg(feature = "grpc-tonic")]
 use crate::{
     exporter::tonic::{HasTonicConfig, TonicExporterBuilder},
-    TonicExporterBuilderSet,
+    TonicExporterBuilderSet, OTEL_EXPORTER_OTLP_PROTOCOL_GRPC,
 };
+use crate::{ExporterBuildError, OTEL_EXPORTER_OTLP_PROTOCOL, OTEL_EXPORTER_OTLP_PROTOCOL_DEFAULT};
+
+#[cfg(feature = "http-proto")]
+use crate::OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_PROTOBUF;
+
+#[cfg(feature = "http-json")]
+use crate::OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_JSON;
 
 #[cfg(any(feature = "http-proto", feature = "http-json"))]
 use crate::{
@@ -126,6 +132,24 @@ impl SpanExporter {
         SpanExporterBuilder::default()
     }
 
+    /// Builds a default [SpanExporter] based on the value of `OTEL_EXPORTER_OTLP_PROTOCOL`.
+    pub fn build_default(self) -> Result<SpanExporter, ExporterBuildError> {
+        match std::env::var(OTEL_EXPORTER_OTLP_PROTOCOL)
+            .unwrap_or(OTEL_EXPORTER_OTLP_PROTOCOL_DEFAULT.to_string())
+            .as_str()
+        {
+            #[cfg(feature = "grpc-tonic")]
+            OTEL_EXPORTER_OTLP_PROTOCOL_GRPC => Self::builder().with_tonic().build(),
+            #[cfg(feature = "http-proto")]
+            OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_PROTOBUF => Self::builder().with_http().build(),
+            #[cfg(feature = "http-json")]
+            OTEL_EXPORTER_OTLP_PROTOCOL_HTTP_JSON => Self::builder().with_http().build(),
+            other => Err(ExporterBuildError::InternalFailure(format!(
+                "Invalid {OTEL_EXPORTER_OTLP_PROTOCOL}: {other}"
+            ))),
+        }
+    }
+
     #[cfg(any(feature = "http-proto", feature = "http-json"))]
     pub(crate) fn from_http(client: crate::exporter::http::OtlpHttpClient) -> Self {
         SpanExporter {