----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5894/#review9053 -----------------------------------------------------------
Ship it! Ship It! - Gordon Sim On July 10, 2012, 8:26 p.m., Alan Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5894/ > ----------------------------------------------------------- > > (Updated July 10, 2012, 8:26 p.m.) > > > Review request for qpid and Gordon Sim. > > > Description > ------- > > > If authentication and ACL are enabled on brokers in a HA cluster, backup > brokers > are unable to replicate even if --ha-username etc. are set to a user with all > priviledges. > > Need to correct this, and update the documentation explaining what > permissions are required to run a secure HA cluster. > > > Diffs > ----- > > /trunk/qpid/cpp/src/qpid/broker/ConnectionHandler.cpp 1359232 > /trunk/qpid/cpp/src/qpid/broker/Link.h 1359232 > /trunk/qpid/cpp/src/qpid/ha/Backup.cpp 1359232 > /trunk/qpid/cpp/src/qpid/ha/BrokerReplicator.h 1359232 > /trunk/qpid/cpp/src/qpid/ha/BrokerReplicator.cpp 1359232 > /trunk/qpid/cpp/src/tests/ha_tests.py 1359232 > /trunk/qpid/doc/book/src/cpp-broker/Active-Passive-Cluster.xml 1359232 > /trunk/qpid/tools/src/py/qpid-ha 1359232 > > Diff: https://reviews.apache.org/r/5894/diff/ > > > Testing > ------- > > make check, new unit test > > > Thanks, > > Alan Conway > >
