Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-26 Thread Tomas Babej
On Thursday 25 of July 2013 15:53:23 Jakub Hrozek wrote: On Thu, Jul 25, 2013 at 03:39:59PM +0200, Tomas Babej wrote: On Thursday 25 of July 2013 09:30:22 Jan Cholasta wrote: On 25.7.2013 09:11, Petr Spacek wrote: On 25.7.2013 09:03, Alexander Bokovoy wrote: On Thu, 25 Jul 2013, Petr

Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-26 Thread Jan Cholasta
On 26.7.2013 11:29, Tomas Babej wrote: After some investigation I decided the correct approach here is to scream at the debug level only, when referral is being ignored. We cannot guide ourselves by the ldap.OPT_REFFERALS option of the underlying connection simply because even if referral

Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-26 Thread Tomas Babej
On Friday 26 of July 2013 12:16:42 Jan Cholasta wrote: On 26.7.2013 11:29, Tomas Babej wrote: After some investigation I decided the correct approach here is to scream at the debug level only, when referral is being ignored. We cannot guide ourselves by the ldap.OPT_REFFERALS option of

Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-26 Thread Martin Kosek
On 07/26/2013 12:23 PM, Tomas Babej wrote: On Friday 26 of July 2013 12:16:42 Jan Cholasta wrote: On 26.7.2013 11:29, Tomas Babej wrote: After some investigation I decided the correct approach here is to scream at the debug level only, when referral is being ignored. We cannot guide

Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-26 Thread Tomas Babej
On Friday 26 of July 2013 13:03:25 Martin Kosek wrote: On 07/26/2013 12:23 PM, Tomas Babej wrote: On Friday 26 of July 2013 12:16:42 Jan Cholasta wrote: On 26.7.2013 11:29, Tomas Babej wrote: After some investigation I decided the correct approach here is to scream at the debug level

Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-26 Thread Martin Kosek
On 07/26/2013 01:19 PM, Tomas Babej wrote: On Friday 26 of July 2013 13:03:25 Martin Kosek wrote: On 07/26/2013 12:23 PM, Tomas Babej wrote: On Friday 26 of July 2013 12:16:42 Jan Cholasta wrote: On 26.7.2013 11:29, Tomas Babej wrote: After some investigation I decided the correct approach

Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-25 Thread Petr Spacek
On 24.7.2013 22:18, Tomas Babej wrote: Hi, When converting the result obtained by python-ldap library, we need to skip unresolved referral entries, since they cannot be converted. https://fedorahosted.org/freeipa/ticket/3814 I'm not sure if a simple 'skip it' approach is the right one.

Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-25 Thread Alexander Bokovoy
On Thu, 25 Jul 2013, Petr Spacek wrote: On 24.7.2013 22:18, Tomas Babej wrote: Hi, When converting the result obtained by python-ldap library, we need to skip unresolved referral entries, since they cannot be converted. https://fedorahosted.org/freeipa/ticket/3814 I'm not sure if a simple

Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-25 Thread Petr Spacek
On 25.7.2013 09:03, Alexander Bokovoy wrote: On Thu, 25 Jul 2013, Petr Spacek wrote: On 24.7.2013 22:18, Tomas Babej wrote: Hi, When converting the result obtained by python-ldap library, we need to skip unresolved referral entries, since they cannot be converted.

Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-25 Thread Alexander Bokovoy
On Thu, 25 Jul 2013, Petr Spacek wrote: On 25.7.2013 09:03, Alexander Bokovoy wrote: On Thu, 25 Jul 2013, Petr Spacek wrote: On 24.7.2013 22:18, Tomas Babej wrote: Hi, When converting the result obtained by python-ldap library, we need to skip unresolved referral entries, since they cannot

Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-25 Thread Jan Cholasta
On 25.7.2013 09:11, Petr Spacek wrote: On 25.7.2013 09:03, Alexander Bokovoy wrote: On Thu, 25 Jul 2013, Petr Spacek wrote: On 24.7.2013 22:18, Tomas Babej wrote: Hi, When converting the result obtained by python-ldap library, we need to skip unresolved referral entries, since they cannot be

Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-25 Thread Tomas Babej
On Thursday 25 of July 2013 09:30:22 Jan Cholasta wrote: On 25.7.2013 09:11, Petr Spacek wrote: On 25.7.2013 09:03, Alexander Bokovoy wrote: On Thu, 25 Jul 2013, Petr Spacek wrote: On 24.7.2013 22:18, Tomas Babej wrote: Hi, When converting the result obtained by python-ldap library,

Re: [Freeipa-devel] [PATCH 0081] Skip referrals when converting LDAP result to LDAPEntry

2013-07-25 Thread Jakub Hrozek
On Thu, Jul 25, 2013 at 03:39:59PM +0200, Tomas Babej wrote: On Thursday 25 of July 2013 09:30:22 Jan Cholasta wrote: On 25.7.2013 09:11, Petr Spacek wrote: On 25.7.2013 09:03, Alexander Bokovoy wrote: On Thu, 25 Jul 2013, Petr Spacek wrote: On 24.7.2013 22:18, Tomas Babej wrote: