fix: prevent logging of RejectedExecutionException

closes: fabric8io#6215 Signed-off-by: Steve Hawkins <[email protected]>
SamBarker · Aug 5, 2024 · 6c24a98 · 6c24a98
1 parent 806a918
commit 6c24a98
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 24 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,7 @@
 #### Bugs
 * Fix #6038: Support for Gradle configuration cache
 * Fix #6110: VolumeSource (and other file mode fields) in Octal are correctly interpreted
+* Fix #6215: Suppressing rejected execution exception for port forwarder
 
 #### Improvements
 * Fix #6008: removing the optional dependency on bouncy castle

diff --git a/...c/main/java/io/fabric8/kubernetes/client/dsl/internal/PortForwarderWebsocketListener.java b/...c/main/java/io/fabric8/kubernetes/client/dsl/internal/PortForwarderWebsocketListener.java
@@ -32,6 +32,7 @@
 import java.util.concurrent.Executor;
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
+import java.util.concurrent.RejectedExecutionException;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.function.BooleanSupplier;
 
@@ -75,16 +76,20 @@ public void onOpen(final WebSocket webSocket) {
           if (e instanceof InterruptedException) {
             Thread.currentThread().interrupt();
           }
-          logger.debug("Error while writing client data");
-          if (alive.get()) {
-            clientThrowables.add(e);
-            closeBothWays(webSocket, 1001, "Client error");
-          }
+          clientError(webSocket, "writing client data", e);
         }
       });
     }
   }
 
+  private void clientError(final WebSocket webSocket, String operation, Exception e) {
+    if (alive.get()) {
+      logger.debug("Error while " + operation, e);
+      clientThrowables.add(e);
+      closeBothWays(webSocket, 1001, "Client error");
+    }
+  }
+
   @Override
   public void onMessage(WebSocket webSocket, String text) {
     logger.debug("{}: onMessage(String)", LOG_PREFIX);
@@ -125,27 +130,27 @@ public void onMessage(WebSocket webSocket, ByteBuffer buffer) {
     } else {
       // Data
       if (out != null) {
-        serialExecutor.execute(() -> {
-          try {
-            while (buffer.hasRemaining()) {
-              int written = out.write(buffer); // channel byte already skipped
-              if (written == 0) {
-                // out is non-blocking, prevent a busy loop
-                Thread.sleep(50);
+        try {
+          serialExecutor.execute(() -> {
+            try {
+              while (buffer.hasRemaining()) {
+                int written = out.write(buffer); // channel byte already skipped
+                if (written == 0) {
+                  // out is non-blocking, prevent a busy loop
+                  Thread.sleep(50);
+                }
               }
+              webSocket.request();
+            } catch (IOException | InterruptedException e) {
+              if (e instanceof InterruptedException) {
+                Thread.currentThread().interrupt();
+              }
+              clientError(webSocket, "forwarding data to the client", e);
             }
-            webSocket.request();
-          } catch (IOException | InterruptedException e) {
-            if (e instanceof InterruptedException) {
-              Thread.currentThread().interrupt();
-            }
-            if (alive.get()) {
-              clientThrowables.add(e);
-              logger.debug("Error while forwarding data to the client", e);
-              closeBothWays(webSocket, 1002, PROTOCOL_ERROR);
-            }
-          }
-        });
+          });
+        } catch (RejectedExecutionException e) {
+          // just ignore 
+        }
       }
     }
   }