- KOReader version: 2024.11
- Device: all
Issue
Looking at my kosync instance, I noticed that it is possible to create usernames which leading/trailing whitespaces. I don't think that is intentional (or is it?). I could trim the usernames on the server, but I think it would be better to adjust the validate function in the kosync plugin.
One disadvantage is that existing (in my opinion incorrect) user names would become invalid.
(Another one is that since validate() is also used for passwords, it won't be possible to have whitespaces in passwords.)
What do you think? In my opinion, it should not be allowed to have whitespaces in usernames (and passwords).
Steps to reproduce
- Go to Progress Sync -> Register / Login
- Provide a username which a leading or trailing whitespace
- The username will be accepted.
Issue
Looking at my kosync instance, I noticed that it is possible to create usernames which leading/trailing whitespaces. I don't think that is intentional (or is it?). I could trim the usernames on the server, but I think it would be better to adjust the validate function in the kosync plugin.
One disadvantage is that existing (in my opinion incorrect) user names would become invalid.
(Another one is that since validate() is also used for passwords, it won't be possible to have whitespaces in passwords.)
What do you think? In my opinion, it should not be allowed to have whitespaces in usernames (and passwords).
Steps to reproduce