Bug #49478 for DBD-LDAP: more than two 'and' clause in the same query lead to an incorrect filter

Mon Sep 07 05:11:37 2009 korsani [...] free.fr - Ticket created

Subject:

more than two 'and' clause in the same query lead to an incorrect filter

When you make a query like that : select cn,sn from base where sn=mySn and givenName=myGn and company=myComp the resulting ldap request is (&((&(givenName=myGn)(sn=mySn)(company=myComp)))) which is an incorrect filter

Mon Sep 07 19:56:07 2009 turnerjw784 [...] yahoo.com - Taken

Mon Sep 07 19:57:53 2009 turnerjw784 [...] yahoo.com - Correspondence added

On Mon Sep 07 05:11:37 2009, KORSANI wrote: Show quoted text

> When you make a query like that : > select cn,sn from base where sn=mySn and givenName=myGn and company=myComp > > the resulting ldap request is > (&((&(givenName=myGn)(sn=mySn)(company=myComp)))) > > which is an incorrect filter

Yes, this indeed is busted! Problem is in sub parse_expression. I think I have a fix for it just about ready, and will release a new version shortly, Thanks, Jim Turner

Mon Sep 07 19:57:54 2009 The RT System itself - Status changed from 'new' to 'open'

Fri Sep 11 11:48:18 2009 korsani [...] free.fr - Correspondence added

Subject:	Re: [rt.cpan.org #49478] more than two 'and' clause in the same query lead to an incorrect filter
Date:	Fri, 11 Sep 2009 17:47:55 +0200
To:	bug-DBD-LDAP [...] rt.cpan.org
From:	Gab <korsani [...] free.fr>

It seems to work with version 0.10 I join you a small patch that avoid warnings in some portions of code. I

Download dbd-ldap.patch.gz
application/x-gzip 796b

Message body not shown because it is not plain text.

Thanks to you ! Le 8 sept. 09 à 01:57, Jim Turner via RT a écrit : Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=49478 > > > On Mon Sep 07 05:11:37 2009, KORSANI wrote:

>> When you make a query like that : >> select cn,sn from base where sn=mySn and givenName=myGn and >> company=myComp >> >> the resulting ldap request is >> (&((&(givenName=myGn)(sn=mySn)(company=myComp)))) >> >> which is an incorrect filter

> > Yes, this indeed is busted! Problem is in sub parse_expression. I > think I have a fix for it just about ready, and will release a new > version shortly, > > Thanks, > > Jim Turner >

-- Gab

Mon Sep 14 12:14:31 2009 turnerjw784 [...] yahoo.com - Correspondence added

On Fri Sep 11 11:48:18 2009, KORSANI wrote: Show quoted text

> It seems to work with version 0.10 > I join you a small patch that avoid warnings in some portions of code. > > I

Thanks for the patch! I have applied it to and released new version v0.10, which is now on my website and is making it's way onto CPAN as I write. I am marking this bug resolved. If you find any more issues in the newest version, please open a new ticket. Thanks again! Jim

Mon Sep 14 12:15:16 2009 turnerjw784 [...] yahoo.com - Correspondence added

This issue is now resolved and closed. See last response. Jim

Mon Sep 14 12:15:18 2009 turnerjw784 [...] yahoo.com - Status changed from 'open' to 'resolved'