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.

Sign up for GitHub

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

Jump to bottom

Use SHA256 instead of MD5 for registry_key when data is not displayable #3990

Closed
wants to merge 2 commits into from
Closed

Use SHA256 instead of MD5 for registry_key when data is not displayable #3990

wants to merge 2 commits into from

Conversation

jaym
Copy link
Contributor

@jaym jaym commented Sep 29, 2015

Getting rid of md5 where we dont need it. See #3945 for more info

@lamont-granquist
Copy link
Contributor

I still think we should use hex instead of hashing it, but 👍

@thommay
Copy link
Contributor

thommay commented Sep 29, 2015

👍

@jaym jaym added this to the 12.6.0 milestone Oct 2, 2015
@jaym jaym force-pushed the jdm/registry-key-no-md5 branch from c6c44fa to f2ac70b Compare October 2, 2015 16:03
@lamont-granquist lamont-granquist mentioned this pull request Oct 25, 2015
Merged
@jaym jaym deleted the jdm/registry-key-no-md5 branch January 27, 2016 04:47
@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@tyler-ball tyler-ball

Labels
None yet
Projects
None yet
Milestone
12.6.0
Development

Successfully merging this pull request may close these issues.
5 participants
@jaym @lamont-granquist @thommay @tyler-ball @chef-supermarket