This is an automated email from the ASF dual-hosted git repository.

jbonofre pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/activemq.git


The following commit(s) were added to refs/heads/master by this push:
     new b830894  AMQ-7069 HTTP client don't handle XStream deserialization 
exception
     new f7b0871  Merge pull request #305 from amarkevich/AMQ-7069
b830894 is described below

commit b830894e690d317ba32019c00d7d4db01eadcc5d
Author: amarkevich <[email protected]>
AuthorDate: Wed Oct 10 16:57:49 2018 +0300

    AMQ-7069 HTTP client don't handle XStream deserialization exception
---
 .../java/org/apache/activemq/transport/http/HttpClientTransport.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/activemq-http/src/main/java/org/apache/activemq/transport/http/HttpClientTransport.java
 
b/activemq-http/src/main/java/org/apache/activemq/transport/http/HttpClientTransport.java
index 2480daa..fee69e3 100644
--- 
a/activemq-http/src/main/java/org/apache/activemq/transport/http/HttpClientTransport.java
+++ 
b/activemq-http/src/main/java/org/apache/activemq/transport/http/HttpClientTransport.java
@@ -204,7 +204,7 @@ public class HttpClientTransport extends 
HttpTransportSupport {
                     }
                     stream.close();
                 }
-            } catch (IOException e) {
+            } catch (Exception e) { // handle RuntimeException from unmarshal
                 onException(IOExceptionSupport.create("Failed to perform GET 
on: " + remoteUrl + " Reason: " + e.getMessage(), e));
                 break;
             } finally {

Reply via email to