qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chug Rolke" <cro...@redhat.com>
Subject Review Request 19199: Invalid URL throws illegal exception during client connection creation
Date Thu, 13 Mar 2014 19:14:05 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for qpid and Gordon Sim.

Bugs: QPID-5623

Repository: qpid


This patch intercepts any qpid::Exception traps that are not types::Exception and rethrows
them as messaging::ConnectionError.

The issue was first noted using amqp1.0 as it is handled correctly in amqp0-10.

This code could be moved to other places in amqp1.0 handlers such as ConnectionHandle.cpp:52
and ConnectionContext.cpp:89. However, there the try/catch would be inserted into constructor
initialization lists where it looks ugly and is hard to read.


  trunk/qpid/cpp/src/qpid/messaging/amqp/ConnectionHandle.cpp 1577280 

Diff: https://reviews.apache.org/r/19199/diff/


Linux client/broker from command line.


Chug Rolke

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message