Skip to content

Add LDAP channel binding support#77

Open
moscowchill wants to merge 1 commit intodirkjanm:masterfrom
moscowchill:ldap-channel-binding-v2
Open

Add LDAP channel binding support#77
moscowchill wants to merge 1 commit intodirkjanm:masterfrom
moscowchill:ldap-channel-binding-v2

Conversation

@moscowchill
Copy link

  • Add --ldap-channel-binding CLI option for servers requiring channel binding
  • Implement TLS channel binding using ldap3 >= 2.10
  • Add helpful error message when strongerAuthRequired (result code 8) is received
  • Check for TLS_CHANNEL_BINDING attribute availability before attempting to use it

This allows ldapdomaindump to work with domain controllers that have LDAP channel binding enforced.

Requires: ldap3 >= 2.10 (pip3 install git+https://github.com/cannatag/ldap3.git)

- Add --ldap-channel-binding CLI option for servers requiring channel binding
- Implement TLS channel binding using ldap3 >= 2.10
- Add helpful error message when strongerAuthRequired (result code 8) is received
- Check for TLS_CHANNEL_BINDING attribute availability before attempting to use it

This allows ldapdomaindump to work with domain controllers that have
LDAP channel binding enforced.

Requires: ldap3 >= 2.10 (pip3 install git+https://github.com/cannatag/ldap3.git)
@moscowchill
Copy link
Author

@dirkjanm @spextat0r This is a refresh of #57 - the original PR referenced ly4k's ldap3 fork which is now archived. Channel binding support has since been merged into the official ldap3 repo (cannatag/ldap3#1087) and will be available in ldap3 >= 2.10.
Until 2.10 is released on PyPI, users can install from master:
pip3 install git+https://github.com/cannatag/ldap3.git
Also adds a helpful hint when strongerAuthRequired error is received, pointing users to the --ldap-channel-binding option. Tested against a DC with channel binding enforced.

@Neustradamus
Copy link

@moscowchill: Nice, thanks for your PR about TLS Channel Binding!

Linked to:

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants