Hello,
since my patch about the LDAP JpegPhoto support, someone contacted me off-list, because he needed LDAP authentication.
It seems that it was hardly implemented into the file "rcube_ldap.inc", but the member function bind() is never called. So I merged the member functions connect() and bind(). Here is my diff between svn ver. 367 and my working copy :
http://pedrov.kwain.net/~pedrov/rcube/rcube_367_wkcpy.diff
I'm interested in developing some LDAP aspects in roundcube, but did not get any feedback, so if you think it is useful (or useless, or whatever), let me know ;-)
See you,
Hello Pierre,
it think it would be very useful to have more LDAP aspects implemented.
I few months ago I sent a patch to this list for a more general ldap
search
on email addresses but no one replied to it:
http://lists.roundcube.net/mail-archive/roundcube.dev/2006/06/118/
The original LDAP search doesn't work for me, as users in my directory mostly have more than one email address, so the search results looked strange. This patch doesn't change the default search behavior.
So maybe it would make sense to integrate it in your patch and extend the search function generally to more LDAP attributes like birthday, telephone numbers etc. Most LDAP implementations provide these attributes so why not use them.
Right now the ldap search isn't working anyway in SVN as there seems to be a major rewrite going on: http://lists.roundcube.net/mail-archive/roundcube.dev/2006/09/136/
Hopefully there is some interest in more LDAP functionality.
Regards, Drazen Baic
Hello,
since my patch about the LDAP JpegPhoto support, someone contacted me off-list, because he needed LDAP authentication.
It seems that it was hardly implemented into the file
"rcube_ldap.inc", but the member function bind() is never called. So I merged the member functions connect() and bind(). Here is my diff between svn ver. 367 and my working copy :http://pedrov.kwain.net/~pedrov/rcube/rcube_367_wkcpy.diff
I'm interested in developing some LDAP aspects in roundcube, but did not get any feedback, so if you think it is useful (or useless, or whatever), let me know ;-)
See you,
-- Pierre Mauduit
Hi again,
I few months ago I sent a patch to this list for a more general ldap
search on email addresses but no one replied to it: http://lists.roundcube.net/mail-archive/roundcube.dev/2006/06/118/
I tried to patch my ldapsearchform.inc but not succeeded in, so I did it "in my own way" ; here is my patch :
http://pedrov.kwain.net/~pedrov/rcube/ldapsearchform.diff
The original LDAP search doesn't work for me, as users in my directory mostly have more than one email address, so the search results looked strange. This patch doesn't change the default search behavior.
On my copy, it seems to be working nice. In fact I had not multiple-email contacts but I tried and did not encountered problems. The search works well, and it prints multiple mail addresses whenever it is necessary. Here is a screenshot of what it looks like :
http://pedrov.kwain.net/~pedrov/rcube/rcube_multiple_mail.png
So maybe it would make sense to integrate it in your patch and extend the search function generally to more LDAP attributes like birthday, telephone numbers etc. Most LDAP implementations provide these attributes so why not use them.
Why not ? I think we should focus on the "InetOrgPerson" structure, which must be the most common.
Right now the ldap search isn't working anyway in SVN as there seems to be a major rewrite going on: http://lists.roundcube.net/mail-archive/roundcube.dev/2006/09/136/
Perhaps should I wait a little then, instead of flooding the dev list with patches about LDAP :-)
regards,
-
Drazen Baic -
Pierre Mauduit