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. -- Endi S. Dewata _______________________________________________ Pki-devel mailing list Pki-devel@redhat.com https://www.redhat.com/mailman/listinfo/pki-devel
