[PATCH] IPA: Don't fail the request when BE doesn't find the object

[Jakub Hrozek]

Hi,

see the attached patch. To reproduce, just run 'getent group
[e-mail]' and make sure that with the patch
applied, groups_get_done() or users_get_done() also run and attempt to
delete the nonexistant user or group.

Additionally, there is a variable called user_dom in the codepath which
handles both users and group. Shall I rename it to obj_dom or similar?

[Pavel Reichl]

On 10/29/2014 09:26 PM, Jakub Hrozek wrote:
> Hi,
>
> see the attached patch. To reproduce, just run 'getent group
> [e-mail]' and make sure that with the patch
> applied, groups_get_done() or users_get_done() also run and attempt to
> delete the nonexistant user or group.

ACK
> Additionally, there is a variable called user_dom in the codepath which
> handles both users and group. Shall I rename it to obj_dom or similar?

Renaming sounds OK to me.
> sssd-devel mailing list
> [e-mail]
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

[Jakub Hrozek]

On Thu, Oct 30, 2014 at 04:19:16PM +0100, Pavel Reichl wrote:
> On 10/29/2014 09:26 PM, Jakub Hrozek wrote:
> > Hi,
> >
> > see the attached patch. To reproduce, just run 'getent group
> > [e-mail]' and make sure that with the patch
> > applied, groups_get_done() or users_get_done() also run and attempt to
> > delete the nonexistant user or group.
>
> ACK

* master: 7b5a077bd162cb2340d5d4e9f5dcc0dd00709f1a
> > Additionally, there is a variable called user_dom in the codepath which
> > handles both users and group. Shall I rename it to obj_dom or similar?
>
> Renaming sounds OK to me.

OK, I'll send another patch.
> > sssd-devel mailing list
> > [e-mail]
> > https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
> sssd-devel mailing list
> [e-mail]
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel