fix: default compression to 'none' (open-telemetry#4985)

Zirak · Sep 14, 2024 · 12147ca · 12147ca
1 parent a2c25d0
commit 12147ca
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/experimental/CHANGELOG.md b/experimental/CHANGELOG.md
@@ -14,6 +14,7 @@ All notable changes to experimental packages in this project will be documented
 ### :bug: (Bug Fix)
 
 * fix(sampler-jaeger-remote): fixes an issue where package could emit unhandled promise rejections @Just-Sieb
+* fix(otlp-grpc-exporter-base): default compression to `'none'` if env vars `OTEL_EXPORTER_OTLP_TRACES_COMPRESSION` and `OTEL_EXPORTER_OTLP_COMPRESSION` are falsy @sjvans
 
 ### :books: (Refine Doc)
 

diff --git a/experimental/packages/otlp-grpc-exporter-base/src/util.ts b/experimental/packages/otlp-grpc-exporter-base/src/util.ts
@@ -160,7 +160,8 @@ export function configureCompression(
 
   const envCompression =
     getEnv().OTEL_EXPORTER_OTLP_TRACES_COMPRESSION ||
-    getEnv().OTEL_EXPORTER_OTLP_COMPRESSION;
+    getEnv().OTEL_EXPORTER_OTLP_COMPRESSION ||
+    'none';
 
   if (envCompression === 'gzip') {
     return CompressionAlgorithm.GZIP;