Project

General

Profile

Actions

Defect #22532

closed

Strip whitespace from login on login page

Added by Artur Borisovich about 8 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Accounts / authentication
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

Spaces in login is removed before user account will saved. But when you try to login you made a mistake like whitespace before login or after it will be not you login.


Files


Related issues

Related to Redmine - Defect #27754: Strip whitespace from email addresses on lost password pageClosedGo MAEDA

Actions
Actions #2

Updated by Toshi MARUYAMA almost 8 years ago

  • Category set to Accounts / authentication
Actions #3

Updated by Marius BĂLTEANU over 6 years ago

  • Related to Defect #27754: Strip whitespace from email addresses on lost password page added
Actions #4

Updated by Go MAEDA over 6 years ago

  • Target version set to 3.4.4

Marius BALTEANU wrote in #27754#note-7:

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

I agree with Marius. I think it would be better to deliver this fix along with #27754.
Setting target version to 3.4.4.

Actions #5

Updated by Marius BĂLTEANU over 6 years ago

The patch attached by Artur Borisovich still applies cleanly. I've added a test for it and I think that we can fix this issue.

Actions #6

Updated by Go MAEDA over 6 years ago

LGTM. I will commit this fix within a few days.

Actions #7

Updated by Go MAEDA over 6 years ago

  • Status changed from New to Resolved
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed to the trunk. Thank you.

Actions #8

Updated by Go MAEDA over 6 years ago

  • Subject changed from Spaces are not removed when the login authorization to Strip whitespace from login on login page
  • Status changed from Resolved to Closed

Merged to 3.4-stable.

Actions #9

Updated by Artur Borisovich over 6 years ago

Go MAEDA wrote:

Merged to 3.4-stable.

Nice, thank you.

Actions

Also available in: Atom PDF