Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Strip spaces? #30

Closed
philtr opened this issue Aug 22, 2016 · 4 comments
Closed

Strip spaces? #30

philtr opened this issue Aug 22, 2016 · 4 comments

Comments

@philtr
Copy link
Contributor

philtr commented Aug 22, 2016

@jdtornow Would you be interested in a PR that stripped leading/trailing spaces from usernames/email addresses?

@jdtornow
Copy link
Owner

Hey @philtr that'd be great, if you're interested.

I think this is already happening on emails, but perhaps not on usernames? Worth another look for sure to keep things stored cleanly.

@philtr
Copy link
Contributor Author

philtr commented Aug 22, 2016

Hmm... I had an email address get saved that didn't get processed through that filter (it had capital letters and a space at the end). I guess I need to do some more digging to figure out what happened on my end here.

@philtr
Copy link
Contributor Author

philtr commented Aug 22, 2016

Haha, I just realized that the project in question is using a different auth library.

@philtr philtr closed this as completed Aug 22, 2016
@jdtornow
Copy link
Owner

Ha! No prob :)

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

No branches or pull requests

2 participants