On Fri, May 20, 2016 at 09:23:46PM +0200, Sumit Bose wrote: > Hi, > > this patch allows the extom plugin to lookup users by certificate which > is needed in the case where a IPA client wants to lookup an AD user who > has the certificate stored in AD. To make this work the related patches > I just send to sssd-devel are needed as well. > > Currently the patches miss the change in the required version of SSSD. > since the SSSD patches are not committed. But the patches are needed to > fully test the SSSD patches. I will send a new version with the needed > changes to the minimal SSSD version when the SSSD patches are committed. > > bye, > Sumit
The patch works fine (tested together with the corresponding SSSD patches), so ACK from me. The code also looks good to me, but I'm not sure if reviewing an IPA patch requires something more (CI? Coverity?) -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
