When circ.password_reset_request_requires_matching_email setting is true, email case must match in addition to content

Bug #1491571 reported by Jason Boyer
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Low
Unassigned
2.8
Fix Released
Undecided
Unassigned
2.9
Fix Released
Undecided
Unassigned

Bug Description

Evergreen 2.0+
Others: N/A

When requesting a password reset it is a good idea to require the user to enter their email to prevent annoying messages. If the setting to require the email match is set though, the case must also match. This has no practical application (especially since it's frequently entered by library staff and not the patron themselves) and only serves to prevent otherwise legitimate requests from being sent because users likely don't enter their email IN ALL CAPS, AS LIBRARY CLERKS HAVE BEEN TRAINED SINCE TIME IMMEMORIAL.

A tiny branch appears on the horizon...

Revision history for this message
Jason Boyer (jboyer) wrote :
Revision history for this message
Ben Shum (bshum) wrote :

Works as far as I can tell. Pushed to master and also backported to rel_2_9 and rel_2_8 as a bug fix.

Changed in evergreen:
milestone: 2.next → 2.10-beta
status: New → Fix Committed
Changed in evergreen:
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.