Skip to content

Commit a0f4647

Browse files
authored
Replace OTLP User-Agent spaces with dashes (#5080)
1 parent 1f975b3 commit a0f4647

File tree

4 files changed

+7
-7
lines changed

4 files changed

+7
-7
lines changed

exporters/otlp/common/src/main/java/io/opentelemetry/exporter/internal/otlp/OtlpUserAgent.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
*/
1515
public final class OtlpUserAgent {
1616

17-
private static final String userAgent = "OTel OTLP Exporter Java/" + readVersion();
17+
private static final String userAgent = "OTel-OTLP-Exporter-Java/" + readVersion();
1818

1919
private static String readVersion() {
2020
Properties properties = new Properties();

exporters/otlp/common/src/test/java/io/opentelemetry/exporter/internal/otlp/OtlpUserAgentTest.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ class OtlpUserAgentTest {
1414

1515
@Test
1616
void getUserAgent() {
17-
assertThat(OtlpUserAgent.getUserAgent()).matches("OTel OTLP Exporter Java/1\\..*");
17+
assertThat(OtlpUserAgent.getUserAgent()).matches("OTel-OTLP-Exporter-Java/1\\..*");
1818
}
1919

2020
@Test
@@ -27,6 +27,6 @@ void addUserAgentHeader() {
2727
valueRef.set(value);
2828
});
2929
assertThat(keyRef.get()).isEqualTo("User-Agent");
30-
assertThat(valueRef.get()).matches("OTel OTLP Exporter Java/1\\..*");
30+
assertThat(valueRef.get()).matches("OTel-OTLP-Exporter-Java/1\\..*");
3131
}
3232
}

exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/AbstractGrpcTelemetryExporterTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ void export() {
224224
.satisfies(
225225
req ->
226226
assertThat(req.headers().get("User-Agent"))
227-
.matches("OTel OTLP Exporter Java/1\\..*"));
227+
.matches("OTel-OTLP-Exporter-Java/1\\..*"));
228228
}
229229

230230
@Test

exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/AbstractHttpTelemetryExporterTest.java

+3-3
Original file line numberDiff line numberDiff line change
@@ -248,9 +248,9 @@ void export() {
248248
assertThat(httpRequests)
249249
.singleElement()
250250
.satisfies(
251-
req -> {
252-
assertThat(req.headers().get("User-Agent")).matches("OTel OTLP Exporter Java/1\\..*");
253-
});
251+
req ->
252+
assertThat(req.headers().get("User-Agent"))
253+
.matches("OTel-OTLP-Exporter-Java/1\\..*"));
254254
}
255255

256256
@Test

0 commit comments

Comments
 (0)