fix fabric8io#3818: adding missing throws to launderThrowable

shawkins · Feb 10, 2022 · 146cb94 · 146cb94
1 parent 677884c
commit 146cb94
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,7 @@
 ### 6.0-SNAPSHOT
 
 #### Bugs
+* Fix #3818: adding missing throws to launderThrowable
 
 #### Improvements
 * Fix #3811: Reintroduce `Replaceable` interface in `NonNamespaceOperation`

diff --git a/kubernetes-client-api/src/main/java/io/fabric8/kubernetes/client/utils/HttpClientUtils.java b/kubernetes-client-api/src/main/java/io/fabric8/kubernetes/client/utils/HttpClientUtils.java
@@ -195,7 +195,7 @@ public static void applyCommonConfiguration(Config config, HttpClient.Builder bu
       }
 
     } catch (Exception e) {
-      KubernetesClientException.launderThrowable(e);
+      throw KubernetesClientException.launderThrowable(e);
     }
     HttpClientUtils.createApplicableInterceptors(config, factory).forEach(builder::addOrReplaceInterceptor);
   }

diff --git a/...nt/src/main/java/io/fabric8/kubernetes/client/dsl/internal/core/v1/PodOperationsImpl.java b/...nt/src/main/java/io/fabric8/kubernetes/client/dsl/internal/core/v1/PodOperationsImpl.java
@@ -436,7 +436,7 @@ private InputStream readFile(String source) {
             out.flush();
             out.close();
           } catch (IOException e) {
-            KubernetesClientException.launderThrowable(e);
+            throw KubernetesClientException.launderThrowable(e);
           }
         }
       ).exec("sh", "-c", String.format("cat %s | base64", shellQuote(source)));
@@ -484,7 +484,7 @@ public InputStream readTar(String source) {
             out.flush();
             out.close();
           } catch (IOException e) {
-            KubernetesClientException.launderThrowable(e);
+            throw KubernetesClientException.launderThrowable(e);
           }
         }
       ).exec("sh", "-c", "tar -cf - " + source + "|" + "base64");