Skip to content
This repository has been archived by the owner on Sep 13, 2018. It is now read-only.

Disallow leading or trailing spaces at account creation #38

Open
Ablu opened this issue Mar 23, 2015 · 4 comments
Open

Disallow leading or trailing spaces at account creation #38

Ablu opened this issue Mar 23, 2015 · 4 comments

Comments

@Ablu
Copy link
Member

Ablu commented Mar 23, 2015

This is pretty confusing, especiall with @ commands.

@bjorn
Copy link
Member

bjorn commented Mar 23, 2015

Ah, you mean when the user name should later be typed in an @ command.

I guess we could just trim it server side. If we would not do it server side then the server would have to check for it, which is almost the same thing.

@Ablu
Copy link
Member Author

Ablu commented Mar 23, 2015

that sounds fine to me too. But we still need to make sure we trim it everywhere. So even at login tries... Otherwise that would be confusing i fear.

@bjorn
Copy link
Member

bjorn commented Mar 23, 2015

Of course.

@bjorn
Copy link
Member

bjorn commented Mar 23, 2015

Or well... the trimming on server side is only required when new accounts are created. For all other purposes, trimming on client-side should suffice.

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

No branches or pull requests

2 participants