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

Update the base encoder #87

Open
Th3Mouk opened this issue Oct 5, 2016 · 1 comment
Open

Update the base encoder #87

Th3Mouk opened this issue Oct 5, 2016 · 1 comment

Comments

@Th3Mouk
Copy link
Contributor

Th3Mouk commented Oct 5, 2016

Hello @djoos,

I would make a PR but before, I'd like to have your opinion.

What are you thinking to update the source code and documentation from sha1 encoder to sha512 ? With a default iteration to 5000.

This configuration is the default one of Symfony, see here. This allow to get a basic configuration enough for non critical applications.
@djoos
Copy link
Owner

djoos commented Dec 19, 2016

Hi @Th3Mouk,

thanks for reaching out: yes, that definitely makes sense!

Kind regards,
David

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@djoos @Th3Mouk