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

ValueError('digestmod is required.') in Pytthon3.8 #702

Closed
Unitiser opened this issue Nov 23, 2019 · 5 comments
Closed

ValueError('digestmod is required.') in Pytthon3.8 #702

Unitiser opened this issue Nov 23, 2019 · 5 comments

Comments

@Unitiser
Copy link

Unitiser commented Nov 23, 2019

According to my research, hmac.py was updated in python3.8 and requires the digestmod to be set explicitly. This issue has been addressed in ldap3 2.6.0.

https://github.com/cannatag/ldap3/blob/master/_changelog.txt

On my machine I forced updated that version and it seamed to fix the issue.

Edit :
Today, I did a bit more digging. I was still getting the error from ntlm.py. I identified occurrences of hmac without explicit digestmod in the following files :

./impacket/ntlm.py
./impacket/dcerpc/v5/nrpc.py
./impacket/examples/secretsdump.py
@asolino
Copy link
Collaborator

asolino commented Nov 26, 2019

Hey @Unitiser

Haven't tested impacket with 3.8. Would you mind issuing a PR to address this?

slasyz added a commit to slasyz/impacket that referenced this issue Nov 28, 2019
@slasyz
Copy link
Contributor

slasyz commented Nov 28, 2019

Hello @asolino

I created a PR, could you please check it?

@Unitiser
Copy link
Author

Awesome @slasyz,

Thanks for doing this, I was planning on working on this issue this weekend but you beat me to it! From what I can tell, the only thing missing is the ldap3 version bump.

@slasyz
Copy link
Contributor

slasyz commented Nov 28, 2019

Thank you, @Unitiser

I think that an extended testing should be performed (including updating ldap3 dependency and testing its usage) to guarantee full Python 3.8 compatibility.

@asolino
Copy link
Collaborator

asolino commented Nov 28, 2019

Fixed in #711

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

No branches or pull requests

3 participants