Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for STARTTLS and/or LDAPS #3

Open
fredreichbier opened this issue Feb 7, 2017 · 5 comments
Open

Support for STARTTLS and/or LDAPS #3

fredreichbier opened this issue Feb 7, 2017 · 5 comments

Comments

@fredreichbier
Copy link
Contributor

The following LDAP communication channels may employ TLS:

  • LDAP Proxy<->LDAP Backend: For that, we can use the LDAPClient.startTLS method, but we should check to which extent certificates are validated first.
  • App<->LDAP Proxy: Here, we will probably use a generic solution provided by Twisted. We will need configuration options to configure certificates, though.

We also need to decide if we only want to support STARTTLS or LDAP over SSL (= LDAPS) as well. 9d818be completely disables TLS support for LDAP for now until we have figured out the questions above.

@cornelinux
Copy link
Member

We also need to support LDAPS to communitcate to the backend. LDAP Proxy<->LDAP Backend. Sometimes AD only supports LDAPS not LDAP+STARTTLS. For that we need to support:

  • LDAP
  • LDAPS
  • LDAP+STARTTLS

@fredreichbier
Copy link
Contributor Author

Current state is:

fredreichbier pushed a commit that referenced this issue Apr 11, 2018
@fredreichbier
Copy link
Contributor Author

I added a note on how to use LDAPS for App<->LDAP Proxy in the 3/serve-ldaps branch. Apparently, this is possible with just Twisted's server endpoint syntax, but I haven't checked for anything but self-signed certificates yet.

@fredreichbier
Copy link
Contributor Author

@cornelinux do you think we can merge #41? Then, we would have official LDAPS support for LDAP Proxy <-> LDAP backend and App <-> LDAP Proxy, so I would close this ticket and open a new one for STARTTLS.

cornelinux pushed a commit that referenced this issue May 8, 2018
@bjo81
Copy link

bjo81 commented Sep 27, 2019

Is it possible to disable cert verification for ldaps?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants