-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10658/
-----------------------------------------------------------
Review request for qpid, Alan Conway and Gordon Sim.
Description
-------
Lock down federation link creation to be allowed only by ACL approval. When no
ACL file is specified then no federation links are allowed.
This version is more forgiving than the description in QPID-4631 as no explicit
CREATE LINK rules are required. Simple 'allow all all' ACL rules are sufficient.
ACL files are added to all diagnostics broker instances so that cmake 'make
check' and autotools 'make test' work.
This addresses bug QPID-4631.
https://issues.apache.org/jira/browse/QPID-4631
Diffs
-----
trunk/qpid/cpp/src/qpid/acl/AclConnectionCounter.cpp 1469944
trunk/qpid/cpp/src/qpid/broker/ConnectionHandler.cpp 1469944
trunk/qpid/cpp/src/tests/ha_test.py 1469944
trunk/qpid/cpp/src/tests/run_acl_tests 1469944
trunk/qpid/cpp/src/tests/run_federation_sys_tests 1469944
trunk/qpid/cpp/src/tests/run_federation_tests 1469944
trunk/qpid/cpp/src/tests/sasl_fed 1469944
Diff: https://reviews.apache.org/r/10658/diff/
Testing
-------
The ACL self test is enhanced to show that brokers running without the ACL
module/file loaded cannot create federation links.
Thanks,
Chug Rolke