search

cblgh / cerca

lean forum software
Other
127 stars 18 forks source link

Lowercase usernames #45

Open alexwennerberg opened 10 months ago

alexwennerberg commented 10 months ago

Right now Alex and alex are separate usernames. I think that the most clear approach is to force all names to lowercase. What do you think?

cblgh commented 10 months ago

i think a better solution would be to explore something like COLLATE NOCASE, or equivalent / better query, on user creation e.g. adapting checkUserExists or whatever the function is called

alexwennerberg commented 10 months ago

I think that makes sense to me. right now, a username can be any unicode string of any length, I am thinking of adding some input validation. Here's what I'd add:

limiting to ascii may be an issue for i18n, but allowing for any string causes a number of problem on the web. what do you think?