On Mon, 2012-05-14 at 10:00 +0200, Martin Kosek wrote:
> On Thu, 2012-05-10 at 13:07 +0200, Petr Viktorin wrote:
> > This is the second and likely the next-to-last part of disabling extra
> > command options (after this it's just test fixes and turning the
> > checking on).
> > Part of the work for https://fedorahosted.org/freeipa/ticket/2509
> This patch looks and works OK. I just think you missed a pkey_only
> attribute for aci_find command. pkey_only is being passed to aci_find
> command by selfservice_find and delegation_find commands and it would
> fail in the hardened tests because it is not defined in aci_find command
> as it is not based on LDAPSearch class but crud.Search.
> It just needs to be added to aci_find command and it should be fine.
Petr³ noticed that pkey_only was already explicitly added to
takes_options of aci_find command. The patch is OK then.
ACK. Pushed to master.
Freeipa-devel mailing list