Summary

TLS support for LDAP authentication

Review Request #433 — Created July 1, 2008 and submitted

Information

Owner

djs52

Repository

Review Board SVN (deprecated)

Branch

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

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.

Testing Done

Authentication continues to work here.

Expand All
Collapse All

Ship it!

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!

```
Looks good. Committed as r1391.
```