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
bash-supergenpass: init at 2012-11-02 #29797
Conversation
de79f54
to
9c62c1a
Compare
description = "Bash shell-script implementation of SuperGenPass password generation"; | ||
longDescription = '' | ||
Bash shell-script implementation of SuperGenPass password generation | ||
Usage: ./supergenpass.sh <domain> [ <length> ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks more like a manual then a description. Do you want to keep it that way?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, bash-supergenpass does not have an --help
argument, so I put the usage there. Is there a more appropriate place where to put it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok then.
The choice of the hash algorithm seems a little bit strange to me: https://github.com/lanzz/bash-supergenpass/blob/master/supergenpass.sh#L12 I would have chosen something like bcrypt or PBKDF2 at least. |
@Mic92 yeah, the algorithm is not the best but It's the one used by supergenpass. https://oneshallpass.com/ is probably the ideal solution, but it lacks implementations (eg for android or sailfish or lightweight linux cli), while sgp has them. |
Motivation for this change
Add the bash implementation of supergenpass.com, since no other implementation is present in nixpkgs
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)The only thing I'm not sure of is whether to include or not the dependencies (
openssl
etc) inbuildInputs
.