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

Allow a single word as audit mode "character" #179

Merged
merged 1 commit into from
Feb 14, 2024
Merged

Conversation

NeffIsBack
Copy link
Contributor

Instead of using one character to hide passwords in audit mode you now can also use a single word:
image

Idea from #178

@NeffIsBack NeffIsBack added the enhancement New feature or request label Feb 14, 2024
@NeffIsBack NeffIsBack mentioned this pull request Feb 14, 2024
@NeffIsBack NeffIsBack merged commit fe179b0 into main Feb 14, 2024
2 checks passed
@NeffIsBack NeffIsBack deleted the neff-audit-mode branch February 14, 2024 14:19
@pgormington-r7
Copy link

Thanks @mpgn 🙌

@mpgn
Copy link
Collaborator

mpgn commented Feb 14, 2024

@NeffIsBack did all the work aha

@pgormington-r7
Copy link

pgormington-r7 commented Feb 26, 2024

Just one note, looks like the output is still incorrect for disabled accounts
image

@mpgn
Copy link
Collaborator

mpgn commented Feb 26, 2024

Cc @NeffIsBack

@NeffIsBack
Copy link
Contributor Author

Just one note, looks like the output is still incorrect for disabled accounts

Did you reinstall Netexec? It worked in my side when it got merged

@pgormington-r7
Copy link

This was a fresh install with pipx

@NeffIsBack
Copy link
Contributor Author

👍🏼 I will recheck later

@NeffIsBack NeffIsBack self-assigned this Feb 26, 2024
@mpgn
Copy link
Collaborator

mpgn commented Feb 26, 2024

https://github.com/search?q=repo%3APennyw0rth%2FNetExec%20self.config.get(%27nxc%27%2C%20%27audit_mode%27)&type=code

:)

@NeffIsBack
Copy link
Contributor Author

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.

3 participants