Bug #17334

Password criteria - show message (Your password cannot contain your username) but accept anyway

Added by Marek Hulán 7 months ago. Updated 7 months ago.

Status:Closed
Priority:Normal
Assigned To:Marek Hulán
Category:Web Interface
Target version:-
Difficulty: Bugzilla link:1394798
Found in release: Pull request:https://github.com/theforeman/foreman/pull/4021
Story points-
Velocity based estimate-
Release1.14.0Release relationshipAuto

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1394798

Description of problem:
In my test, I just create one user virtwho and defined the password as virtwho00, in the Password row we can see the message below

Password - Your password cannot contain your username

and in the Verify row

Verify - password match

But after submit, we can see the password was accepted.

Version-Release number of selected component (if applicable):
6.2.4 / Current foreman nightly

How reproducible:
100%

Steps to Reproduce:
1. Create new user
2. Define the password with the username as part of password
3. Click on Submit

Actual results:
Ignored validation message is being displayed

Expected results:
Message Your password cannot contain your username is not being displayed.

Additional info:
I changed the expected results in compare to original BZ report after short discussion with others. I updated the BZ too.

Associated revisions

Revision 64601c8a
Added by Marek Hulán 7 months ago

Fixes #17334 - change password strength messages to imply warning

History

#1 Updated by The Foreman Bot 7 months ago

  • Status changed from New to Ready For Testing
  • Assigned To set to Marek Hulán
  • Pull request https://github.com/theforeman/foreman/pull/4021 added

#2 Updated by Marek Hulán 7 months ago

  • Subject changed from Password criteria - show message (Your password cannot contain your username) but accept anyway to Password criteria - show message (Your password cannot contain your username) but accept anyway
  • Category changed from Authorization to Web Interface

#3 Updated by Anonymous 7 months ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#4 Updated by Dominic Cleal 7 months ago

  • Release set to 1.14.0

Also available in: Atom PDF