Replace OTLP User-Agent spaces with dashes (#5080)

open-telemetry · Jan 5, 2023 · a0f4647 · a0f4647
1 parent 1f975b3
commit a0f4647
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/...ters/otlp/common/src/main/java/io/opentelemetry/exporter/internal/otlp/OtlpUserAgent.java b/...ters/otlp/common/src/main/java/io/opentelemetry/exporter/internal/otlp/OtlpUserAgent.java
@@ -14,7 +14,7 @@
  */
 public final class OtlpUserAgent {
 
-  private static final String userAgent = "OTel OTLP Exporter Java/" + readVersion();
+  private static final String userAgent = "OTel-OTLP-Exporter-Java/" + readVersion();
 
   private static String readVersion() {
     Properties properties = new Properties();

diff --git a/.../otlp/common/src/test/java/io/opentelemetry/exporter/internal/otlp/OtlpUserAgentTest.java b/.../otlp/common/src/test/java/io/opentelemetry/exporter/internal/otlp/OtlpUserAgentTest.java
@@ -14,7 +14,7 @@ class OtlpUserAgentTest {
 
   @Test
   void getUserAgent() {
-    assertThat(OtlpUserAgent.getUserAgent()).matches("OTel OTLP Exporter Java/1\\..*");
+    assertThat(OtlpUserAgent.getUserAgent()).matches("OTel-OTLP-Exporter-Java/1\\..*");
   }
 
   @Test
@@ -27,6 +27,6 @@ void addUserAgentHeader() {
           valueRef.set(value);
         });
     assertThat(keyRef.get()).isEqualTo("User-Agent");
-    assertThat(valueRef.get()).matches("OTel OTLP Exporter Java/1\\..*");
+    assertThat(valueRef.get()).matches("OTel-OTLP-Exporter-Java/1\\..*");
   }
 }
diff --git a/...va/io/opentelemetry/exporter/otlp/testing/internal/AbstractGrpcTelemetryExporterTest.java b/...va/io/opentelemetry/exporter/otlp/testing/internal/AbstractGrpcTelemetryExporterTest.java
@@ -224,7 +224,7 @@ void export() {
         .satisfies(
             req ->
                 assertThat(req.headers().get("User-Agent"))
-                    .matches("OTel OTLP Exporter Java/1\\..*"));
+                    .matches("OTel-OTLP-Exporter-Java/1\\..*"));
   }
 
   @Test

diff --git a/...va/io/opentelemetry/exporter/otlp/testing/internal/AbstractHttpTelemetryExporterTest.java b/...va/io/opentelemetry/exporter/otlp/testing/internal/AbstractHttpTelemetryExporterTest.java
@@ -248,9 +248,9 @@ void export() {
     assertThat(httpRequests)
         .singleElement()
         .satisfies(
-            req -> {
-              assertThat(req.headers().get("User-Agent")).matches("OTel OTLP Exporter Java/1\\..*");
-            });
+            req ->
+                assertThat(req.headers().get("User-Agent"))
+                    .matches("OTel-OTLP-Exporter-Java/1\\..*"));
   }
 
   @Test