Use non-routable loop back address to avoid issues with firewalls

open-telemetry · Jan 3, 2024 · fe14fdb · fe14fdb
1 parent 9a70faf
commit fe14fdb
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/...va/io/opentelemetry/exporter/otlp/testing/internal/AbstractHttpTelemetryExporterTest.java b/...va/io/opentelemetry/exporter/otlp/testing/internal/AbstractHttpTelemetryExporterTest.java
@@ -513,7 +513,7 @@ void connectTimeout() {
     TelemetryExporter<T> exporter =
         exporterBuilder()
             // Connecting to a non-routable IP address to trigger connection error
-            .setEndpoint("http://10.255.255.1")
+            .setEndpoint("http://127.255.255.255")
             .setConnectTimeout(Duration.ofMillis(1))
             .build();
     try {

diff --git a/...er/jdk/src/test/java/io/opentelemetry/exporter/sender/jdk/internal/JdkHttpSenderTest.java b/...er/jdk/src/test/java/io/opentelemetry/exporter/sender/jdk/internal/JdkHttpSenderTest.java
@@ -53,7 +53,8 @@ void setup() throws IOException, InterruptedException {
     sender =
         new JdkHttpSender(
             mockHttpClient,
-            "http://10.255.255.1", // Connecting to a non-routable IP address to trigger connection
+            "http://127.255.255.255", // Connecting to a non-routable IP address to trigger
+            // connection
             // timeout
             null,
             false,

diff --git a/.../src/test/java/io/opentelemetry/exporter/sender/okhttp/internal/RetryInterceptorTest.java b/.../src/test/java/io/opentelemetry/exporter/sender/okhttp/internal/RetryInterceptorTest.java
@@ -149,7 +149,9 @@ void connectTimeout() throws Exception {
     // Connecting to a non-routable IP address to trigger connection error
     assertThatThrownBy(
             () ->
-                client.newCall(new Request.Builder().url("http://10.255.255.1").build()).execute())
+                client
+                    .newCall(new Request.Builder().url("http://127.255.255.255").build())
+                    .execute())
         .isInstanceOf(SocketTimeoutException.class);
 
     verify(isRetryableException, times(5)).apply(any());
@@ -166,7 +168,9 @@ void nonRetryableException() throws InterruptedException {
     // Connecting to a non-routable IP address to trigger connection timeout
     assertThatThrownBy(
             () ->
-                client.newCall(new Request.Builder().url("http://10.255.255.1").build()).execute())
+                client
+                    .newCall(new Request.Builder().url("http://127.255.255.255").build())
+                    .execute())
         .isInstanceOf(SocketTimeoutException.class);
 
     verify(isRetryableException, times(1)).apply(any());