search

anhapu / SDFplay

3 stars 0 forks source link

Email Case Sensitive #11

Closed Jensus closed 10 years ago

Jensus commented 10 years ago

Tach, grade festgestellt das Email Adressen beim unserem Login Case Sensitive behandelt werden. Macht das Sinn? Ich glaube nämlich, dass es beim Provider zB keinen Unterschied macht wie die Email Adresse geschrieben wird, und dementsprechend darf es bei uns auch keinen machen...

ganddev commented 10 years ago

Kann es sein, dass die Validierung durch die Annotation @Email geschieht? Wenn ja lass es doch einfach so, warum groß dazu etwas zu programmieren?

Jensus commented 10 years ago

Denke eher es liegt an der authenticate Methode im Usermodel. Da sie ja auch nur ein .equals für Strings absetzt, findet sie den User nur bei richtiger Groß/Klein Schreibung. An sich ist es ja egal, ist halt nur eine Frage der Usability die wir vllt klären sollten.

ganddev commented 10 years ago

Dann sollte bei dem Vergleich noch ein toLowercase() / bzw. toUpercase() eingefügt werden...Am besten vom Product Owner verabschieden lassen... ;)

IvenZ commented 10 years ago

PO möchte keinen CS Login

Jensus commented 10 years ago

Login ist nun Case Insensitive.