#2120 - catch all exceptions during response parsing

hyperledger-web3j · Dec 16, 2024 · 863e8df · 863e8df
1 parent 6e553a5
commit 863e8df
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,6 +9,7 @@ See [Conventional Commits](https://conventionalcommits.org) for commit guideline
 
 * fixed several code issues found by sonar [#2113](https://github.com/hyperledger/web3j/pull/2113)
 * update GitHub actions versions [#2114](https://github.com/hyperledger/web3j/pull/2114)
+* fixed request parsing exception handling [#2120](https://github.com/hyperledger/web3j/pull/2120)
 
 ### Features
 

diff --git a/core/src/main/java/org/web3j/protocol/websocket/WebSocketService.java b/core/src/main/java/org/web3j/protocol/websocket/WebSocketService.java
@@ -299,7 +299,7 @@ private void processRequestReply(String replyStr, JsonNode replyJson) throws IOE
             }
 
             sendReplyToListener(request, reply);
-        } catch (IllegalArgumentException e) {
+        } catch (Exception e) {
             sendExceptionToListener(replyStr, request, e);
         }
     }
@@ -322,7 +322,7 @@ private void processBatchRequestReply(String replyStr, ArrayNode replyJson) thro
             }
 
             sendReplyToListener(webSocketRequests, new BatchResponse(requests, responses));
-        } catch (IllegalArgumentException e) {
+        } catch (Exception e) {
             sendExceptionToListener(replyStr, webSocketRequests, e);
         }
     }
@@ -375,7 +375,7 @@ private void sendReplyToListener(WebSocketRequest request, Object reply) {
     }
 
     private void sendExceptionToListener(
-            String replyStr, WebSocketRequest request, IllegalArgumentException e) {
+            String replyStr, WebSocketRequest request, Exception e) {
         request.getOnReply()
                 .completeExceptionally(
                         new IOException(