[SSSD] [PATCH] LDAP: Don't abort request if no id mapping domain matches

2014-01-24 Thread Jakub Hrozek
Hi, During further testing, Kaushik found out that requesting an ID that doesn't match any configured ID mapping domain still emits strange error messages: (Wed Jan 22 11:35:58 2014) [sssd[be[sssdad2012.com]]] [ad_account_info_complete] (0x0010): Bug: dp_error is OK on failed request(Wed Jan 22

Re: [SSSD] [PATCH] LDAP: Don't abort request if no id mapping domain matches

2014-01-24 Thread Sumit Bose
On Fri, Jan 24, 2014 at 10:53:02AM +0100, Jakub Hrozek wrote: Hi, During further testing, Kaushik found out that requesting an ID that doesn't match any configured ID mapping domain still emits strange error messages: (Wed Jan 22 11:35:58 2014) [sssd[be[sssdad2012.com]]]

Re: [SSSD] [PATCH] LDAP: Don't abort request if no id mapping domain matches

2014-01-24 Thread Jakub Hrozek
On Fri, Jan 24, 2014 at 11:06:33AM +0100, Sumit Bose wrote: On Fri, Jan 24, 2014 at 10:53:02AM +0100, Jakub Hrozek wrote: Hi, During further testing, Kaushik found out that requesting an ID that doesn't match any configured ID mapping domain still emits strange error messages:

Re: [SSSD] [PATCH] LDAP: Don't abort request if no id mapping domain matches

2014-01-24 Thread Sumit Bose
On Fri, Jan 24, 2014 at 10:53:02AM +0100, Jakub Hrozek wrote: Hi, During further testing, Kaushik found out that requesting an ID that doesn't match any configured ID mapping domain still emits strange error messages: (Wed Jan 22 11:35:58 2014) [sssd[be[sssdad2012.com]]]

Re: [SSSD] [PATCH] LDAP: Don't abort request if no id mapping domain matches

2014-01-24 Thread Jakub Hrozek
On Fri, Jan 24, 2014 at 12:00:13PM +0100, Sumit Bose wrote: ACK. I wonder if you want to fix a copy-and-paste error in the following comment before push the patch? @@ -497,7 +513,19 @@ struct tevent_req *groups_get_send(TALLOC_CTX *memctx, /* Convert the UID to its

Re: [SSSD] [PATCH] LDAP: Don't abort request if no id mapping domain matches

2014-01-24 Thread Sumit Bose
On Fri, Jan 24, 2014 at 01:21:36PM +0100, Jakub Hrozek wrote: On Fri, Jan 24, 2014 at 12:00:13PM +0100, Sumit Bose wrote: ACK. I wonder if you want to fix a copy-and-paste error in the following comment before push the patch? @@ -497,7 +513,19 @@ struct tevent_req

Re: [SSSD] [PATCH] LDAP: Don't abort request if no id mapping domain matches

2014-01-24 Thread Jakub Hrozek
On Fri, Jan 24, 2014 at 01:27:51PM +0100, Sumit Bose wrote: On Fri, Jan 24, 2014 at 01:21:36PM +0100, Jakub Hrozek wrote: On Fri, Jan 24, 2014 at 12:00:13PM +0100, Sumit Bose wrote: ACK. I wonder if you want to fix a copy-and-paste error in the following comment before push the