Thanks, Pushed to master.
On Wed, 2016-04-20 at 15:23 -0500, Endi Sukma Dewata wrote: > On 4/19/2016 9:47 PM, Ade Lee wrote: > > Some comments inline, although most of this was discussed on #irc. > > > > I have added two additional patches which are to be applied on top > > of 258=293. > > > > 294: This patch fixes the problems identified in this review. In > > particular: > > > > Review comments addressed: > > 1. when archiving or generating keys, realm is checked > > 2. when no plugin is found for a realm, access is denied. > > 3. rename mFoo to foo for new variables. > > 4. add chaining of exceptions > > 5. remove attributes from KeyArchivalRequest etc. when realm > > is > > null > > 6. Add more detail to denial in BasicGroupAuthz > > > > 295 - Adds the ability for authz plugins to support multiple > > realms. > > In particular, the authorize() command has been extended to > > allow > > the realm to be passed in, and the ACL plugins have been > > modified > > to account for the realm. > > > > Please review, > > ACK. > _______________________________________________ Pki-devel mailing list [email protected] https://www.redhat.com/mailman/listinfo/pki-devel
