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.

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

Passwortlänge ist auf 20 Zeichen begrenzt #36

Closed
schmijos opened this issue Jan 8, 2015 · 4 comments
Closed

Passwortlänge ist auf 20 Zeichen begrenzt #36

schmijos opened this issue Jan 8, 2015 · 4 comments
Assignees
Milestone

Comments

@schmijos
Copy link
Member

schmijos commented Jan 8, 2015

Das Passwort-Element hat eine Begrenzung auf 20 Zeichen. Dieses Limit sollten wir aufheben (stammt wahrscheinlich aus der Zeit als wir noch md5 verwendet hatten):
https://github.com/noproblan/npl-website/blob/develop/library/Npl/Form/Element/Password.php

Siehe: https://blog.knut.me/warum-laengenbeschraenkungen-fuer-passwoerter-nichts-taugen.html
Über die Passwörter bei uns:

@schmijos schmijos added the bug label Jan 8, 2015
@schmijos schmijos self-assigned this Jan 8, 2015
@schmijos schmijos added this to the v2.0.3 milestone Jan 8, 2015
@dbrgn
Copy link

dbrgn commented Jan 8, 2015

SHA1 ist nicht so toll, besser bcrypt oder PBKDF2. Die sind gemacht um Passwörter zu speichern.

Gibt zB hier eine Library: https://github.com/rchouinard/phpass (Habe sie selber nie getestet.)

@schmijos
Copy link
Member Author

schmijos commented Jan 8, 2015

Vorerst wird es sha1 bleiben. Laut Wikipedia gibt es (noch) keine bekannte Kollision.

Btw: Ich vermute dass die Begrenzung von 20 Zeichen eingeführt wurde als wir noch md5 verwendet hatten. Die Begrenzung von 40 in der Datenbank sollte also auch aufgehoben werden wenn man für etwas stärkeres als sha1 offen sein will.

@Chuvisco88
Copy link
Member

Zum Thema bcrypt gibt es auch etwas für das eingesetzte ZF1 -> http://stackoverflow.com/questions/15432568/bcrypt-with-zend-framework-1-12-x

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

No branches or pull requests

3 participants