Make LDAP over TLS optional

[adroll]

Make LDAP over TLS optional

[frangarciam]

Created #9 for this.

It's still probably not a good idea to do this but our ldap config could be more flexible, so we might spend some time on it in the future.

[copumpkin]

Agreed that it's not a great idea in general, but I think your approach of putting "insecure" prominently in the name helps alleviate some of the concerns around accidental use. I'm less concerned about the network attacks that TLS guards against in an AWS environment, though.

[copumpkin]

Even if you don't merge the PR (thanks for making it, by the way), I'd still suggest clarifying the documentation, since I tried giving it a :389 endpoint and it just says "connection reset by peer" 😄

[zylad]

#9 has been merged. Also docs has been updated. Thanks for reporting!

[copumpkin]

Thank you!

On Mar 10, 2015, at 07:56, Dominik Zyla [email protected] wrote:

Closed #8.

—
Reply to this email directly or view it on GitHub.