Defect #27754

Strip whitespace from email addresses on lost password page

Added by Felix Schäfer 7 days ago. Updated 5 days ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Go MAEDA% Done:

0%

Category:Accounts / authentication
Target version:3.4.4
Resolution:Fixed Affected version:

Description

Sometimes people will not notice that they copied and pasted or inadvertently entered a whitespace when entering their email address in the lost password page. This currently leads to no password recovery being sent as the whitespaces will prevent a user from being found.

27754.patch Magnifier (1.44 KB) Felix Schäfer, 2017-12-07 12:25


Related issues

Related to Redmine - Defect #22532: Strip whitespace from login on login page Closed

Associated revisions

Revision 17078
Added by Go MAEDA 6 days ago

Fix: Strip whitespace from email addresses on lost password page (#27754).

Patch by Felix Schäfer.

Revision 17079
Added by Go MAEDA 5 days ago

Merged r17078 to 3.4-stable (#27754).

History

#1 Updated by Felix Schäfer 7 days ago

The attached patch provides a test and strips whitespace from the entered email address.

#2 Updated by Jan from Planio www.plan.io 7 days ago

  • Target version set to Candidate for next minor release

#3 Updated by Go MAEDA 7 days ago

  • Tracker changed from Feature to Defect
  • Target version changed from Candidate for next minor release to 3.4.4

Sometimes users say "Lost password page is not working". I understood the reason by seeing this patch!

Setting target version to 3.4.4.

#4 Updated by Go MAEDA 6 days ago

  • Assignee set to Go MAEDA

#5 Updated by Go MAEDA 6 days ago

  • Status changed from New to Resolved

#6 Updated by Marius BALTEANU 6 days ago

  • Related to Defect #22532: Strip whitespace from login on login page added

#7 Updated by Marius BALTEANU 6 days ago

I think that we should fix also the related issue (#22532) because is almost on the same topic.

#8 Updated by Go MAEDA 5 days ago

  • Status changed from Resolved to Closed
  • Resolution set to Fixed

Commited in the trunk and merged to 3.4-stable. Thank you for detecting and fixing this issue.

Also available in: Atom PDF