Review Board

beta

TLS support for LDAP authentication

Submitted
Updated 4 months, 2 weeks ago

Dan Sheridan Reviewers
reviewboard
None Review Board SVN
Add TLS support to the LDAP authentication backend. Enable by adding LDAP_TLS=True to settings_local.py. Assumes appropriate settings (certificate, etc.) in /etc/ldap/ldap.conf.
Authentication continues to work here.
Ship it!
Posted 4 months, 2 weeks ago (July 2nd, 2008, 12:17 p.m.)
If your server supports LDAP over SSL, you can just use "ldaps://..." in the LDAP_URI.  That's probably a simpler route for most people.

If the server doesn't listen on port 636, then you have to use a normal "ldap://" on port 389 and call start_tls_s().  Your change looks fine to me for that case.
Ship it!
Posted 4 months, 2 weeks ago (July 7th, 2008, 11:49 p.m.)
Looks good. Committed as r1391.