openerp-community team mailing list archive
-
openerp-community team
-
Mailing list archive
-
Message #00492
Re: [Merge] lp:~ibeardslee/openobject-addons/users_ldap-tls into lp:~openerp-community/openobject-addons/stefan-therp_lp794584
Hi Ian,
you cannot replace simple_bind_s(dn, password) with anything else, because this is where the user is authenticated ;-)
In the other instance of simple_bind_s, we are authenticating the configured LDAP user for querying the database. You should definitely not change this around.
I remain by my suggestion that you change res_company_ldap.connect() as follows:
connection = ldap.initialize(uri)
if conf['ldap_tls']:
connection.start_tls_s()
return connection
Cheers,
Stefan.
--
https://code.launchpad.net/~ibeardslee/openobject-addons/users_ldap-tls/+merge/71131
Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-addons/stefan-therp_lp794584.
References