Since I was the bad boy who submitted the patch I probably should get it out of CVS, or at least make someone get it out.
src/ejabberd_auth_ldap.erl:38
eldap:start_link("ejabberd_bind", LDAPServers, 389, RootDN, Password),
should be changed back to:
src/ejabberd_auth_ldap.erl:38
eldap:start_link("ejabberd_bind", LDAPServers, 389, "", ""),
Why? The first time I introduced the RootDN, Password fields I was under the impression the bind was used for binding the server to LDAP, this is not true. The ejabberd_bind is used to bind for authetication of a user, and the provided DN and Password should be enough, no need for an allready binded connection.
I inform Alexey about this po
I inform Alexey about this post :-)
--
sander
Aditional change to auth LDAP
Aditional change for fix ldap auth:
src/ejabberd_auth_ldap.erl:98
case eldap:search("ejabberd", [{base, Base},
should be changed to:
src/ejabberd_auth_ldap.erl:98
case eldap:search("ejabberd_bind", [{base, Base},
this use "ejabberd_bind" to bind for search user.