From fa69db3ca76f6297be40f5424ca4a09e440c0003 Mon Sep 17 00:00:00 2001 From: BrianDevC <116743850+BrianDevC@users.noreply.github.com> Date: Tue, 1 Nov 2022 10:32:02 +0000 Subject: [PATCH] iss-4535 Altered some catch statements to catch exceptions rather than throwables for sonar #4535 --- .../internal/core/v1/PodOperationsImpl.java | 20 +++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/kubernetes-client/src/main/java/io/fabric8/kubernetes/client/dsl/internal/core/v1/PodOperationsImpl.java b/kubernetes-client/src/main/java/io/fabric8/kubernetes/client/dsl/internal/core/v1/PodOperationsImpl.java index e3ee180a970..fa873acc0f4 100644 --- a/kubernetes-client/src/main/java/io/fabric8/kubernetes/client/dsl/internal/core/v1/PodOperationsImpl.java +++ b/kubernetes-client/src/main/java/io/fabric8/kubernetes/client/dsl/internal/core/v1/PodOperationsImpl.java @@ -185,8 +185,8 @@ public Loggable withLogWaitTimeout(Integer logWaitTimeout) { public PortForward portForward(int port, ReadableByteChannel in, WritableByteChannel out) { try { return new PortForwarderWebsocket(httpClient, this.context.getExecutor()).forward(getResourceUrl(), port, in, out); - } catch (Throwable t) { - throw KubernetesClientException.launderThrowable(t); + } catch (Exception e) { + throw KubernetesClientException.launderThrowable(e); } } @@ -194,8 +194,8 @@ public PortForward portForward(int port, ReadableByteChannel in, WritableByteCha public LocalPortForward portForward(int port) { try { return new PortForwarderWebsocket(httpClient, this.context.getExecutor()).forward(getResourceUrl(), port); - } catch (Throwable t) { - throw KubernetesClientException.launderThrowable(t); + } catch (Exception e) { + throw KubernetesClientException.launderThrowable(e); } } @@ -203,8 +203,8 @@ public LocalPortForward portForward(int port) { public LocalPortForward portForward(int port, int localPort) { try { return new PortForwarderWebsocket(httpClient, this.context.getExecutor()).forward(getResourceUrl(), port, localPort); - } catch (Throwable t) { - throw KubernetesClientException.launderThrowable(t); + } catch (Exception e) { + throw KubernetesClientException.launderThrowable(e); } } @@ -274,8 +274,8 @@ public ExecWatch exec(String... command) { try { URL url = getExecURLWithCommandParams(actualCommands); return setupConnectionToPod(url.toURI()); - } catch (Throwable t) { - throw KubernetesClientException.launderThrowable(forOperationType("exec"), t); + } catch (Exception e) { + throw KubernetesClientException.launderThrowable(forOperationType("exec"), e); } } @@ -284,8 +284,8 @@ public ExecWatch attach() { try { URL url = getAttachURL(); return setupConnectionToPod(url.toURI()); - } catch (Throwable t) { - throw KubernetesClientException.launderThrowable(forOperationType("attach"), t); + } catch (Exception e) { + throw KubernetesClientException.launderThrowable(forOperationType("attach"), e); } }