Skip to content

Reconsider the use of md5() and uniqid() #21

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

Open
cmb69 opened this issue Jul 26, 2024 · 0 comments
Open

Reconsider the use of md5() and uniqid() #21

cmb69 opened this issue Jul 26, 2024 · 0 comments

Comments

@cmb69
Copy link
Member

cmb69 commented Jul 26, 2024

This has been triggered by the discussion regarding the vote to deprecate md5(), sha1() and uniqid().

We're using md5() and uniqid() for caching purposes, what might cause collisions, so perhaps increasing the entropy would be a good idea. To avoid overly long filenames, we might use base32 encoding (base64 encoding isn't suitable for case insensitive file systems), instead of base16.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant