--On October 28, 2011 2:36:54 PM -0700 Quanah Gibson-Mount
<qua...@zimbra.com> wrote:
--On October 28, 2011 4:14:02 PM +0000 Viktor Dukhovni
<postfix-us...@dukhovni.org> wrote:
FREE_RESULT)); + rc = ldap_parse_result(ld, res, &err, 0, 0, 0, 0,
This should be dict_ldap->ld rather than just "ld". ;)
Working on testing this now.
It correctly rejects if the password is wrong now:
postmap: dict_ldap_connect: Actual Protocol version used is 3.
postmap: dict_ldap_connect: Binding to server
ldap://zre-ldap002.eng.vmware.com:389 with dn
uid=zmpostfix,cn=appaccts,cn=zimbra
postmap: warning: dict_ldap_connect: Unable to bind to server
ldap://zre-ldap002.eng.vmware.com:389 with dn
uid=zmpostfix,cn=appaccts,cn=zimbra: 49 (Invalid credentials)
It correctly succeeds if the password is correct:
postmap: dict_ldap_connect: Binding to server
ldap://zre-ldap002.eng.vmware.com:389 with dn
uid=zmpostfix,cn=appaccts,cn=zimbra
postmap: dict_ldap_connect: Successful bind to server
ldap://zre-ldap002.eng.vmware.com:389 with dn
uid=zmpostfix,cn=appaccts,cn=zimbra
postmap: dict_ldap_connect: Cached connection handle for LDAP source
/opt/zimbra/conf/ldap-transport.cf
postmap: dict_ldap_lookup: /opt/zimbra/conf/ldap-transport.cf: Searching
with filter
(&(|(zimbraMailDeliveryAddress=testus...@zre-ldap002.eng.vmware.com)(zimbraDomainName=testus...@zre-ldap002.eng.vmware.com))(zimbraMailStatus=enabled))
postmap: dict_ldap_get_values[1]: Search found 1 match(es)
postmap: dict_ldap_get_values[1]: search returned 1 value(s) for requested
result attribute zimbraMailTransport
postmap: dict_ldap_get_values[1]: Leaving dict_ldap_get_values
postmap: dict_ldap_lookup: Search returned
lmtp:zre-ldap002.eng.vmware.com:7025
lmtp:zre-ldap002.eng.vmware.com:7025
postmap: dict_ldap_close: Closed connection handle for LDAP source
/opt/zimbra/conf/ldap-transport.cf
So I would say the patch looks good!
--Quanah
--
Quanah Gibson-Mount
Principal Software Engineer
Zimbra, Inc
--------------------
Zimbra :: the leader in open source messaging and collaboration
