----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18423/#review35522 -----------------------------------------------------------
Ship it! Looks good! Thanks for your patience Pavel! - Gordon Sim On Feb. 26, 2014, 12:22 p.m., Pavel Moravec wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18423/ > ----------------------------------------------------------- > > (Updated Feb. 26, 2014, 12:22 p.m.) > > > Review request for qpid, Chug Rolke and Gordon Sim. > > > Bugs: QPID-5561 and QPID-5565 > https://issues.apache.org/jira/browse/QPID-5561 > https://issues.apache.org/jira/browse/QPID-5565 > > > Repository: qpid > > > Description > ------- > > This patch fixes both: > > QPID-5561: C++ Broker queue creation ACL checks against wrong limit values > QPID-5565: [C++ broker] ACL queue create rules to take zero as unlimited value > > The patch: > - passes to ACL persistency-related / paging queue options only if the queue > is marked as durable / with paging > - fills proper default values > > > Potential issues: > - I had to copy constant value "static const u_int32_t defJrnlFileSizePgs = > 24" and also "defNumJrnlFiles = 8" as there is no simple way (known to me) to > test if legacy store module is loaded > - anyway as legacy store is not supposed to be changed (due to > linearstore), I see this a acceptable > - ACL line like: > > acl allow all create queue filemaxcountupperlimit=32 > > is _not_ matched for a transient queue. That differs from current behaviour, > anyway I see it as an improvement rather (as is, the above ACL is meaningless > for transient queues that dont have filecount limit applicable) > > > Diffs > ----- > > /trunk/qpid/cpp/src/qpid/broker/Broker.cpp 1571980 > /trunk/qpid/cpp/src/qpid/broker/QueueFactory.h 1571980 > /trunk/qpid/cpp/src/qpid/broker/QueueFactory.cpp 1571980 > /trunk/qpid/cpp/src/tests/acl.py 1571980 > > Diff: https://reviews.apache.org/r/18423/diff/ > > > Testing > ------- > > - different variants of use cases (esp. from the JIRAs) passed > - automated tests passed > > > Thanks, > > Pavel Moravec > >
