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

Can't change password or email address because current password is "invalid" #1184

Closed
durin42 opened this issue Jan 10, 2024 · 2 comments
Closed

Comments

@durin42
Copy link

durin42 commented Jan 10, 2024

Describe the bug
I can't change my password or email address in https://www.commafeed.com/#/app/settings, I think because my password is old enough it predates the current requirement to have a special character in the password.

To Reproduce
Steps to reproduce the behavior:

  1. Have an old account with an "invalid" password that has no special character
  2. Go to https://www.commafeed.com/#/app/settings
  3. Attempt to change a setting that requires the user's password
  4. Save
  5. See error

Expected behavior
My changes are accepted.

Screenshots
If applicable, add screenshots to help explain your problem.

Screenshot 2024-01-10 at 1 17 13 PM

Environment (please complete the following information):

  • CommaFeed version (or "commafeed.com"): CommaFeed version 4.0.0 (e90b373).
  • Browser [e.g. chrome, firefox]: Firefox
  • Device [e.g. desktop, mobile]: macOS 14.2.1

Additional context
This account is very old, and I haven't changed the password in a goodly while. I'm speculating that the issue is the lack of a special character, because the username/password are accepted by the login form, I just can't change my password (or add an email to my account) for the above-mentioned reason.

@Athou
Copy link
Owner

Athou commented Jan 10, 2024

Could you try again now please? CommaFeed was verifying the "current password" field of the form by matching it with your email but you didn't have any because email was not mandatory before. I changed to code to make the match with usernames since all users have one.

@durin42
Copy link
Author

durin42 commented Jan 10, 2024

That appears to have fixed it. Thanks!

@durin42 durin42 closed this as completed Jan 10, 2024
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