[wp-trac] [WordPress Trac] #42766: Issue in update password From admin side and login ith same password
WordPress Trac
noreply at wordpress.org
Mon Feb 17 17:49:50 UTC 2020
#42766: Issue in update password From admin side and login ith same password
-------------------------------------+-------------------------------------
Reporter: ronakganatra | Owner: adamsilverstein
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: 5.4
Component: Users | Version:
Severity: normal | Resolution:
Keywords: good-first-bug has- | Focuses: ui, javascript,
patch has-unit-tests needs- | administration, performance
testing |
-------------------------------------+-------------------------------------
Changes (by nrqsnchz):
* keywords:
good-first-bug has-patch has-unit-tests needs-testing needs-design-
feedback
=> good-first-bug has-patch has-unit-tests needs-testing
Comment:
Discussed today during design triage meeting in Slack:
https://wordpress.slack.com/archives/C02S78ZAL/p1581961322460700
>Do we want a password of only spaces/space-like characters (e.g. tabs
maybe if pasted in) to be allowed?
Not sure if this can be answered by the design team. This seems more like
a security or compliance issue.
>If the second case, then when creating the user, should the feedback to
the user change at all, since they did enter a non-empty password and the
error message would be "password empty"?
If the password is rejected because it contains empty spaces, and these
are forbidden, then yes, the error messages should be clear and indicate
why the password is being rejected.
>Should the updating user page be able to update (e.g. click the button)
if the only thing updated is the password, and it's an invalid password
I don't think so. If an error has occurred, this should be communicated to
the user instead of falsely making them believe the operation was a
success.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/42766#comment:20>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list