← Back to team overview

openerp-community team mailing list archive

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