Skip to content
This repository has been archived by the owner on Nov 24, 2023. It is now read-only.

Trim usernames #225

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Trim usernames #225

wants to merge 1 commit into from

Conversation

goneall
Copy link
Collaborator

@goneall goneall commented Aug 20, 2022

When a user registers, resets, or logs in with a user with leading or trailing spaces, the app may reject the request due to a mismatch with with is in the database.

Signed-off-by: Gary O'Neall [email protected]

Signed-off-by: Gary O'Neall <[email protected]>
@goneall
Copy link
Collaborator Author

goneall commented Aug 20, 2022

IMPORTANT: There are 3 users with trailing spaces as of the time of this pull request. Before deploying this PR, the database needs to be update to remove any leading or trailing spaces from all usernames stored in the database - otherwise these users will never be able to login again.

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

Successfully merging this pull request may close these issues.

1 participant