Skip to content

Add Support for impacket-secretsdump result#3

Open
pasya1912 wants to merge 1 commit into
strikoder:mainfrom
pasya1912:patch-1
Open

Add Support for impacket-secretsdump result#3
pasya1912 wants to merge 1 commit into
strikoder:mainfrom
pasya1912:patch-1

Conversation

@pasya1912

Copy link
Copy Markdown

Add argument -from-ntlm to parse ntlm hashes from impacket-secretsdump result.

During one of the lab that i did i came across a SAM, ntds.dit, and SYSTEM file where i can use impacket-secretsdump to get NTLM hashes of users.
the output will be formatted like this:
[Username]:[RID]:[LM Hash]:[NTLM Hash]:::

Manually parsing the username and NT hash is takes time so i made this, i think this should be implemented in CredSpray

anyway thanks dude this wrapper bash script is really great.

Add argument -from-ntlm to parse ntlm hashes from impacket-secretsdump result.
@pasya1912

Copy link
Copy Markdown
Author

To be transparent:
im using AI to modify the script , i dont mind if you reject it because its from ai.
One thing for sure is i tested it and it works just fine.

@strikoder

Copy link
Copy Markdown
Owner

I don't mind using AI if it improves the functionality of the code, so no worries :D
Addtionally, tnx for being upfront about it. However, I need some time to review everything before merging, as I always double-check changes carefully. A delayed merge doesn't necessarily mean the PR has been rejected.

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