You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by rg...@apache.org on 2016/04/03 16:42:20 UTC

svn commit: r1737588 - /qpid/java/trunk/broker-plugins/amqp-1-0-protocol/src/main/java/org/apache/qpid/server/protocol/v1_0/SendingLink_1_0.java

Author: rgodfrey
Date: Sun Apr  3 14:42:19 2016
New Revision: 1737588

URL: http://svn.apache.org/viewvc?rev=1737588&view=rev
Log:
QPID-7181 : Ensure all TokenMgrError is caught along with other parsing exceptions when attempting to construct a JMS filter

Modified:
    qpid/java/trunk/broker-plugins/amqp-1-0-protocol/src/main/java/org/apache/qpid/server/protocol/v1_0/SendingLink_1_0.java

Modified: qpid/java/trunk/broker-plugins/amqp-1-0-protocol/src/main/java/org/apache/qpid/server/protocol/v1_0/SendingLink_1_0.java
URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/amqp-1-0-protocol/src/main/java/org/apache/qpid/server/protocol/v1_0/SendingLink_1_0.java?rev=1737588&r1=1737587&r2=1737588&view=diff
==============================================================================
--- qpid/java/trunk/broker-plugins/amqp-1-0-protocol/src/main/java/org/apache/qpid/server/protocol/v1_0/SendingLink_1_0.java (original)
+++ qpid/java/trunk/broker-plugins/amqp-1-0-protocol/src/main/java/org/apache/qpid/server/protocol/v1_0/SendingLink_1_0.java Sun Apr  3 14:42:19 2016
@@ -63,6 +63,7 @@ import org.apache.qpid.amqp_1_0.type.tra
 import org.apache.qpid.exchange.ExchangeDefaults;
 import org.apache.qpid.filter.SelectorParsingException;
 import org.apache.qpid.filter.selector.ParseException;
+import org.apache.qpid.filter.selector.TokenMgrError;
 import org.apache.qpid.server.consumer.ConsumerImpl;
 import org.apache.qpid.server.filter.FilterManager;
 import org.apache.qpid.server.filter.JMSSelectorFilter;
@@ -155,7 +156,7 @@ public class SendingLink_1_0 implements
 
                             actualFilters.put(entry.getKey(), entry.getValue());
                         }
-                        catch (ParseException | SelectorParsingException e)
+                        catch (ParseException | SelectorParsingException | TokenMgrError e)
                         {
                             Error error = new Error();
                             error.setCondition(AmqpError.INVALID_FIELD);
@@ -285,7 +286,7 @@ public class SendingLink_1_0 implements
 
                                 actualFilters.put(entry.getKey(), entry.getValue());
                             }
-                            catch (ParseException | SelectorParsingException e)
+                            catch (ParseException | SelectorParsingException | TokenMgrError e)
                             {
                                 Error error = new Error();
                                 error.setCondition(AmqpError.INVALID_FIELD);



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org