Adding crypt to pwgen output #12
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Why only sha256crypt? And may I also suggest to add a "brake" and sleep for 1s before it prints out the result?! Just my personal 2cents... |
New -e flag to output proposed crypt string with password. Signed-off-by: ed neville <ed@s5h.net>
|
On 2023-08-22 12:24-0700, Matthias Kraft wrote:
Why only sha256crypt? And may I also suggest to add a "brake" and
sleep for 1s before it prints out the result?! Just my personal
2cents...
Only because that was what I found useful. Updated. What was the reason
for the 1s sleep?
Ed
|
|
To not create a tool that could easily be used to create thousands of encrypted passwords in a second. Granted, people who know what they are doing have that already, but "script-kiddies" really don't need that. And while thinking of it again ... what is the use-case anyway? |
|
On 2023-08-27 02:24-0700, Matthias Kraft wrote:
To not create a tool that could easily be used to create thousands of encrypted passwords in a second. Granted, people who know what they are doing have that already, but "script-kiddies" really don't need that.
I might be missing something, why is the crypt bad here?
And while thinking of it again ... what is the use-case anyway?
I frequently run useradd followed by passwd to set the pass. Sometimes
introducing typos or whitespace. Taking the crypt form to use with
useradd -p or usermod -p is more helpful I believe.
Ed
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New -e flag to output proposed crypt string with password.