[Freeipa-devel] [freeipa PR#575][comment] IPA certauth plugin

2017-03-27 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/575 Title: #575: IPA certauth plugin dkupka commented: """ master: * da880decfedc66f9d0d2734dcb86c23a8866f603 ipa-kdb: add ipadb_fetch_principals_with_extra_filter() * c4156041feb9c48598427ad59e43313b9c7327bb IPA certauth plugin ipa-4-5: *

[Freeipa-devel] [freeipa PR#575][comment] IPA certauth plugin

2017-03-27 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/575 Title: #575: IPA certauth plugin dkupka commented: """ I've tested the patches and it worked as expected. Once CI successfully finishes I'll ACK it. """ See the full comment at

[Freeipa-devel] [freeipa PR#575][comment] IPA certauth plugin

2017-03-23 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/575 Title: #575: IPA certauth plugin abbra commented: """ The code LGTM. Once updated SSSD is added to freeipa-master copr, let's see what CI says. Authentication indicators' handling would need to be added in a separate PR once certmap rules

[Freeipa-devel] [freeipa PR#575][comment] IPA certauth plugin

2017-03-14 Thread sumit-bose
URL: https://github.com/freeipa/freeipa/pull/575 Title: #575: IPA certauth plugin sumit-bose commented: """ I updated the code to reflect the latest changes in the interface from https://github.com/krb5/krb5/pull/610. """ See the full comment at

[Freeipa-devel] [freeipa PR#575][comment] IPA certauth plugin

2017-03-13 Thread sumit-bose
URL: https://github.com/freeipa/freeipa/pull/575 Title: #575: IPA certauth plugin sumit-bose commented: """ This patch depends on https://github.com/SSSD/sssd/pull/192 (SSSD's certmap library) and https://github.com/krb5/krb5/pull/610 (MIT Kerberos certauth plugin support) """ See the full