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

[enhancement] Add NTLMv1 scan option #68

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

AdrianVollmer
Copy link
Contributor

See #67. I included portions of Responder's code with minimal modifications. This means that Coercer must be GPL licensed.

In this approach, we monkeypatch Responder's SaveToDb function to modify control_structure accordingly.

When setting --stop-on-ntlm-auth in scan mode, coercer stops scanning a target completely upon receiving an SMB connection with NTLM authentication. This is useful if we want to find DCs supporting NTLMv1.

This PR should be taken as a proposal. Happy to discuss details!

Minor modifications have been made so we are able to load this as a
module. Also, all top-level commands in `Responder.py` have been
removed, like the printing of the banner
@p0dalirius p0dalirius self-assigned this Nov 9, 2023
@p0dalirius p0dalirius added the enhancement New feature or request label Nov 9, 2023
@p0dalirius p0dalirius changed the title Ntlm scan [enhancement] Add NTLMv1 scan option Nov 9, 2023
@p0dalirius
Copy link
Owner

p0dalirius commented Nov 9, 2023

Hey @AdrianVollmer,

This is a very good idea, I'll look into it in January 2024!

Best regards,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants