Separate username and password validation on Log In screen

Bug #491651 reported by George
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Open Library
Fix Released
Medium
Anand Chitipothu

Bug Description

Hi Anand,

On the Upstream Log In screen, if I enter either an incorrect username or password, I need to be informed of which of them is incorrect, or if they are both incorrect.

I just tested the login, and entered an incorrect password with my normal username. The error message said: "Invalid username or password"

1) If the username is incorrect, we need to display a message that reads "The username you entered isn't in the Open Library system. Please try again?"

2) If the password is incorrect, we need to display a message that reads "That password seems incorrect. Please try again, or, if you've forgotten it, have your _password reset_."

3) If both username and password are incorrect, just show both messages. (Though, ideally, we would provide a third note that reads "Hmm... We don't recognize the username or the password you supplied. Please try again?")

* Note, errors should read as complete sentences, with punctuation
* If you'd like me to write things for you, that's absolutely no problem.

George (george-archive)
Changed in openlibrary:
assignee: nobody → Anand Chitipothu (anandology)
importance: Undecided → High
milestone: none → upstream
importance: High → Medium
status: New → Confirmed
Revision history for this message
Edward Betts (edwardbetts) wrote :

The code is in /vendor/infogami/infogami/infobase/server.py

The logic needs to change to check if a user exists before trying to login.

Revision history for this message
Anand Chitipothu (anandology) wrote :

We can first check existence of the user by testing /people/$username page.

Revision history for this message
Anand Chitipothu (anandology) wrote :

handled case#1 and #2.

Case#3 doesn't make sense because it is not possible to have wrong username and right password.

Error messages are specified in:
http://upstream.openlibrary.org/templates/errors.tmpl

Revision history for this message
George (george-archive) wrote :

Sweet! Thank you very much for getting to this - it's a pet peeve relieved!

George (george-archive)
Changed in openlibrary:
status: Confirmed → Fix Committed
Changed in openlibrary:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.