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

Couple items from your cryptpwutil wishlist #2

Merged
merged 2 commits into from Jan 31, 2015

Conversation

Projects
None yet
2 participants
@doyleyoung
Contributor

doyleyoung commented Jan 31, 2015

Thank you for your feedback on my previous pull request, I learned a lot about Dist::Zilla and it's available functionality as a result. This pull request includes a couple items from your wishlist.

** WISHLIST [2015-01-25 Min] cryptpwutil: (as separate function?) an option to return detailed information about a crypted string
instead of just recognizing its type, we can also parse and return its salt,
header, variant (e.g. in MD5-CRYPT there's an apache variant), or other
information.

** WISHLIST [2015-01-25 Min] cryptpwutil: improve documentation
explain in first paragraph of description that this module is geared towards generation and recognition of unix passwords (/etc/shadow), that's why crypt() tries several methods but check that each method is supported by your system before returning the result.
more pointers/links to each crypt type should a user want to find out more.

doyleyoung added some commits Jan 31, 2015

Add git_detail command
Add git_detail command that return crypt type, header, salt and hash values in a
string
Add associated documentation and testing
Updated documentation
Added a DESCRIPTION section describing functionality
Added links to hash algorithm details in crypt_type function description

perlancar added a commit that referenced this pull request Jan 31, 2015

Merge pull request #2 from doyleyoung/cpanpr
Couple items from your cryptpwutil wishlist

@perlancar perlancar merged commit e69c281 into perlancar:master Jan 31, 2015

@perlancar

This comment has been minimized.

Show comment
Hide comment
@perlancar

perlancar Jan 31, 2015

Owner

I've released 0.11 which includes features from this PR. I've done some modification: 1) crypt_detail() is merged into crypt_type() as an $detail option; 2) the detailed information is returned as a hash instead

Thanks!

Owner

perlancar commented Jan 31, 2015

I've released 0.11 which includes features from this PR. I've done some modification: 1) crypt_detail() is merged into crypt_type() as an $detail option; 2) the detailed information is returned as a hash instead

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment