Trailing whitespace in username leads to invalid credentials

[saschadoemer]

Expected Behavior

As a user, I would expect trailing spaces to be removed in case I copied them with trailing spaces from messages, mails or something else.

Current Behavior

The current version uses the username as it is. Therefore, trailing spaces are used as part of the login and cause an invalid login, which just leads to highlighted fields without any further information.

Leads to the following outcome:

The trailing space is hard to notice, and the UI is giving no hint.

Additional information

Thanks for the awesome work ❤️

[cesnietor]

thanks for reporting @saschadoemer, PRs are welcome.

[saschadoemer]

I'll try, let me check the code 😃

[changchaishi]

Hi, I would like to work on this. Is that okay?

Based on what I observed on other websites (for instance, Github), it accepts trailing space and handles it on the backend server.
The change will be like strings.TrimRight(accessKey, " ") to get the console credentials.

I will file the PR if this is ok. Thanks.

[ramondeklein]

Please go ahead and file a PR. I would both trim spaces both at the beginning and end. It's not very likely that someone would use spaces in the beginning or end of it's username/password. And if you do, you we're asking for trouble 😉