-
Notifications
You must be signed in to change notification settings - Fork 3.8k
[LDAP] Add custom cbt_value #1977
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
Conversation
|
@zblurx thanks for the PR. I think it would be nice to have an example to test signing/channel binding, not only because it would be helpful as an info gathering tool, but also to validate these changes as well. |
|
@zblurx any update :) ? |
|
We are waiting changes adding an example that leverages these changes, as it was mentioned above |
|
Hey @anadrianmanrique @mpgn sorry for the delay, here is the example script.
|
|
AH I'm gonna do the same for MSSQL then! |
- make channel_bidinng_value variable public
|
ok, I pushed changes related to code review in order to have this merged ASAP as we are closing 0.13 release. |
examples/CheckLDAPStatus.py
Outdated
| # Login failed (wrong credentials). test if we get an error with an existing, but wrong CBT -> When supported | ||
| elif str(e).find("data 52e") >= 0: | ||
| ldap_connection = LDAPConnection(url=ldap_url) | ||
| new_cbv = bytearray(ldap_connection._LDAPConnection__channel_binding_value) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
either define channel_biding_value as a public member in LDAPConnection or implement getter/setter methods

Hey guys ! This simple PR allows for channel binding customization. This helps us a lot on NetExec to check for channel binding status. If you want, I can add a custom example script in Impacket to do LDAP Signing / Channel binding check. Let me know !