-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1608/
-----------------------------------------------------------
Review request for qpid and rajith attapattu.
Summary
-------
This patch changes the 0-10 code path to create the SASL callback handler using
the CallbackHandlerRegistry. This allows the 0-10 code path to support SASL
mechanisms requiring other callback handlers, such as CRAM-MD5-HASHED. Support
for the sasl_mechs client connection option has been retained and now applies
to the 0-8..0-9-1 code paths too.
If the user *specifies* a sasl_mechs client connection option the behaviour of
the code is unchanged from the previous version: it restricts the list of SASL
mechanisms in use.
If the user *does not specify* a sasl_mechs client connection option, the old
code used a hardcoded PLAIN default. This is no longer the case. Now the
client will use the first SASL mechanism from the list
CallbackHandlerRegistry.properties that is also available on the server.
Removed dead code and strengthen unit tests.
This addresses bug QPID-3415.
https://issues.apache.org/jira/browse/QPID-3415
Diffs
-----
/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQConnectionDelegate_0_10.java
1160136
/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/handler/ConnectionStartMethodHandler.java
1160136
/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/security/CallbackHandlerRegistry.java
1160136
/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/security/CallbackHandlerRegistry.properties
1160136
/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/ClientDelegate.java
1160136
/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/Connection.java
1160136
/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/ConnectionSettings.java
1160136
/trunk/qpid/java/common/src/test/java/org/apache/qpid/transport/ConnectionTest.java
1160136
/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/unit/client/connection/ConnectionTest.java
1160136
/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/unit/message/UTF8Test.java
1160136
Diff: https://reviews.apache.org/r/1608/diff
Testing
-------
Improved unit testing. Run java, cpp and cpp.ssl profiles. I am not able to
test GSSAPI locally.
Thanks,
Keith
