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.

Sign up for GitHub

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

Jump to bottom

Password reset input field normalization #4135

Closed
aapeliv opened this issue Feb 6, 2022 · 5 comments · May be fixed by Couchers-org/web-frontend#398
Closed

Password reset input field normalization #4135

aapeliv opened this issue Feb 6, 2022 · 5 comments · May be fixed by Couchers-org/web-frontend#398
Labels
0.kind bug report This shouldn't be happening. 1.topic frontend 3.skill good first issue

Comments

@aapeliv
Copy link
Member

aapeliv commented Feb 6, 2022

Seems the password field doesn't lowercase usernames, etc. so on mobile this often causes issues.

See https://community.couchers.org/t/issue-with-password-reset-for-couchers-app/1741

Also as discussed somewhere, backend should return meaningful error messages; username enumeration is easy enough already.
@mketer1
Copy link

mketer1 commented Mar 22, 2022

Hi! I'm fairly new to open source contribution but I would love to give this a try. I may need some guidance/assistance if that is okay. If the issue is still open, could I be assigned to it?

@AndrewFarley AndrewFarley transferred this issue from Couchers-org/couchers Jun 2, 2022
@Roshanjossey
Copy link
Contributor

@mketer1, were you able to setup the app locally?

@Roshanjossey Roshanjossey mentioned this issue Jan 14, 2024
Open
8 tasks
@aapeliv aapeliv transferred this issue from Couchers-org/web-frontend Apr 20, 2024
@jesseallhands
Copy link
Contributor

@aapeliv I think this is fixed as I couldn't reproduce, but can you confirm?

@bakeiro
Copy link
Contributor

bakeiro commented Oct 14, 2024

I think this task is done... (couldn't reproduce it back then)

@aapeliv
Copy link
Member Author

aapeliv commented Oct 14, 2024

Yeah thought we fixed it!

@aapeliv aapeliv closed this as completed Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
0.kind bug report This shouldn't be happening. 1.topic frontend 3.skill good first issue
Projects
Couchers Engineering 💻
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: add email type attribute to email input fields Roshanjossey/web-frontend
6 participants
@aapeliv @Roshanjossey @schradert @bakeiro @mketer1 @jesseallhands