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

Winkerberos Support, ldap3 MD4 Bug Fix #17

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from
Open

Conversation

ehlewis
Copy link

@ehlewis ehlewis commented Oct 10, 2024

Added dependencies required for BloodHound to utilize Kerberos.
Fixed an issue in the version of ldap3 that was zipped in the repo that occurred in NTLM handling when MD4 is not already available on the system. If this was the case, ldap3 would try to load the MD4 module from Crypto which is a dependency we did not have. The adjusted version will load MD4 from Cryptodome, which we are already have. This is a known and supported fix as per cannatag/ldap3#1051

naksyn and others added 9 commits March 11, 2024 12:47
This updated version of LDAP fixes an issue in NTLM handling where MD4 may not be available already on the system. If this was the case it would try to load the MD4 module from Crypto which is a dependency we did not have installed. This version will load MD4 from Cryptodome, which we are already installing in memory
Non-standard file, its not there in the ldap3 repo currently, but I'm unsure if it is expected by this application
ldap3 was failing to import due to missing winkerberos or gssapi. gssapi would not need to be build for the specific arch, but would require a dependency of MIT Kerberos
need 3 - not 2
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.

3 participants