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

Removed swear words from passwords #931

Merged
merged 9 commits into from
Feb 13, 2024
Merged

Conversation

molangning
Copy link
Contributor

@molangning molangning commented Nov 25, 2023

Files included in commits: desweared passwords, swear word remover script and source files

As noted by OP, this can be used in academic settings. However, the real world has curse words in passwords so dont use this to crack passwords hash you dumped from your engagement(obv)

This is an implementation of issue #677

@molangning molangning changed the title implemention of issue #677 implementation of issue #677 Nov 26, 2023
@g0tmi1k
Copy link
Collaborator

g0tmi1k commented Nov 26, 2023

I would opt to have the script and list of words to be removed,
I wouldn't commit the result of the script - as people can do that themselves

@g0tmi1k g0tmi1k self-assigned this Nov 26, 2023
@g0tmi1k g0tmi1k added the enhancement Enhancement label Nov 26, 2023
@molangning
Copy link
Contributor Author

so just the passwords without curse words?

@molangning
Copy link
Contributor Author

the script is in there in case anyone want to generate their own custom list/remove more/ add to seclists

@molangning
Copy link
Contributor Author

I would opt to have the script and list of words to be removed, I wouldn't commit the result of the script - as people can do that themselves

I re-read this and I still dont understand you haha

@molangning molangning changed the title implementation of issue #677 Removed swear words from passwords Nov 26, 2023
@g0tmi1k
Copy link
Collaborator

g0tmi1k commented Nov 27, 2023

The files in .bin and Miscellaneous/list-of-swear-words/ keep.

However, the stuff in Passwords/Common-Credentials-without-curse-words could you remove

@molangning
Copy link
Contributor Author

molangning commented Nov 27, 2023

im leaving a readme in bin folder

my reasoning is that people still need to know what the scripts are for

@molangning
Copy link
Contributor Author

Done that

@g0tmi1k
Copy link
Collaborator

g0tmi1k commented Feb 13, 2024

Great stuff @molangning

@g0tmi1k g0tmi1k merged commit 4b0dc7f into danielmiessler:master Feb 13, 2024
1 check passed
@molangning molangning deleted the patch-3 branch February 14, 2024 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants