Re: [evolution-patches] Re-sent: Patch for 318227: Change LDAP search filter [mail/addressbook]

["Carsten Guenther" <Carsten Guenther scalix com>]

Sure it does, that's the whole point :-)

Maybe we are talking about two different things here. I am talking about 
mailing-lists that are resolved on the server (and that are maintained 
by the server administrator) and where the list itself has an email 
address. For example at Scalix we have eng scalix com with alle the 
engineers being members, but the email address eng scalix com is all you 
need to write to all the members.

So, obviously the query "objectclass=person" does not cut it because 
mailing-lists have objectclass "distributionList" or something 
different, depending of the product. But by using "mail=*" you get every 
entry that is adressable, mailing-lists included. BTW, that's how most 
other clients (Thunderbird for exmaple) do it as well.

I am aware that this does not entirely fix the issue "Evolution does not 
deal with LDAP groups", but it is a one-line fix that fixes some of the 
issues and one that can be applied to 2.4 as well (that's what we 
actually need).

Carsten

Chris Toshok wrote:
> Meaning that when sent the mail doesn't go to all the members of the
> ldap mailing list.  Or does it?
>
> On Mon, 2005-10-10 at 12:53 -0700, Carsten Guenther wrote:
>   
> > What do you mean? Changing the search filter to "(mail=*)" allows me
> > to search and autocomplete mailing-lists. Autocompletion was the main
> > problem for us.
> >
> > Chris Toshok wrote: 
> >     
> > > Not really..  what happens when you autocomplete against them?  The
> > > point is just adding (mail=*) to the query doesn't fix the actual
> > > problem, which is that evo doesn't deal with mailing lists on ldap
> > > servers.
> > >
> > > On Mon, 2005-10-10 at 12:40 -0700, Carsten Guenther wrote:
> > >   
> > >       
> > > > With my patch they would show up in addressbook searches. Only the lists 
> > > > email address will be displayed though, no membership information, but 
> > > > that is fine.
> > > >
> > > > Chris Toshok wrote:
> > > >     
> > > >         
> > > > > are mailing lists from ldap reflected as contact lists in the UI.  Seems
> > > > > like you'd want that as well.
> > > > >
> > > > > On Mon, 2005-10-10 at 12:12 -0700, Carsten Guenther wrote:
> > > > >   
> > > > >       
> > > > >           
> > > > > > Can I commit this?
> > > > > >
> > > > > > --
> > > > > > Attached patch fixes 318227. The LDAP search filter used was limited to
> > > > > > entries with objectclass "person" which prevented searching for
> > > > > > mailing-lists, a problem many people have. The patch changes the search
> > > > > > filter such that it looks for everything that has the attribute "mail"
> > > > > > set ("mail=*"). That's basically what Thunderbird and other clients do
> > > > > > as well.
> > > > > >
> > > > > > I think bug 311884 still is valid but can be fixed t a later point.
> > > > > >
> > > > > > Please review for commit to gnome-2-12 and head.
> > > > > >
> > > > > > Thanks,
> > > > > > Carsten
> > > > > >
> > > > > >
> > > > > > _______________________________________________
> > > > > > Evolution-patches mailing list
> > > > > > Evolution-patches gnome org
> > > > > > http://mail.gnome.org/mailman/listinfo/evolution-patches
> > > > > >     
> > > > > >         
> > > > > >             
> > > _______________________________________________
> > > Evolution-patches mailing list
> > > Evolution-patches gnome org
> > > http://mail.gnome.org/mailman/listinfo/evolution-patches
> > >   
> > >