Adding UncheckedIOException to onFailure handler

jack-berg · Dec 19, 2023 · 8c3e8f7 · 8c3e8f7
1 parent bfbf15a
commit 8c3e8f7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/...sender/jdk/src/main/java/io/opentelemetry/exporter/sender/jdk/internal/JdkHttpSender.java b/...sender/jdk/src/main/java/io/opentelemetry/exporter/sender/jdk/internal/JdkHttpSender.java
@@ -12,7 +12,6 @@
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.io.OutputStream;
-import java.io.UncheckedIOException;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.net.http.HttpClient;
@@ -127,8 +126,8 @@ public void send(
                 () -> {
                   try {
                     return sendInternal(marshaler);
-                  } catch (IOException | UncheckedIOException e) {
-                    onError.accept(e);
+                  } catch (IOException e) {
+                    throw new UncheckedIOException(e);
                   }
                 },
                 executorService)

diff --git a/...http/src/main/java/io/opentelemetry/exporter/sender/okhttp/internal/OkHttpHttpSender.java b/...http/src/main/java/io/opentelemetry/exporter/sender/okhttp/internal/OkHttpHttpSender.java
@@ -146,7 +146,7 @@ public byte[] responseBody() throws IOException {
                                   return body.bytes();
                                 }
                               });
-                        } catch (UncheckedIOException e) {
+                        } catch (Throwable e) {
                           onError.accept(e);
                         }
                       }