tumblr / collins

groovy kind of love
tumblr.github.com/collins
Apache License 2.0
570 stars 99 forks source link

Inconsistencies in LDAPS document #587

Closed lukegriffith closed 6 years ago

lukegriffith commented 6 years ago

Under configuration. Authentication states that LDAP over SSL is not supported, yet further down in the configuration options for LDAP there is a Boolean for SSL.

Is it supported or not? And can the documentation be updated to be consistent.

defect commented 6 years ago

Sorry for the slow response. Collins does support ldaps, so the part of the docs you're referencing are indeed incorrect. I'll cut a PR to update the docs.

As an aside, I think it still would be wise to run a LDAP slave on the box next to collins :) Nothing worse when you have a big outage and you can't reach your inventory management database.

defect commented 6 years ago

Closed in #588