Bug 988342

Summary: Should not report info when set password with create form and should report create error info when mismath password and confirm password
Product: [Retired] oVirt Reporter: wanghui <huiwa>
Component: ovirt-nodeAssignee: Ryan Barry <rbarry>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: unspecifiedCC: acathrow, cshao, gouyang, hadong, jboggs, leiwang, mburns, mgoldboi, ovirt-bugs, ovirt-maint, yaniwang, ycui, yeylon
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: ovirt-node-3.0.3 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-11-28 11:58:40 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
error infos none

Description wanghui 2013-07-25 11:24:20 UTC
Created attachment 778222 [details]
error infos

Description of problem:
It should not report info when setting password with creatly form. And it should report creatly error info when mismatch password with confirm password.

Version-Release number of selected component (if applicable):
ovirt-node-iso-3.1.0-0.999.999.20130723112131git21ad9e6.2916.fc19.iso
ovirt-node-3.1.0-0.999.20130723112131git21ad9e6.fc19.noarch

How reproducible:
100%

Steps to Reproduce:
1. Clear install ovirt-node.
2. Set Password in Security page as follows.
   Password:       redhat
3. Set confirm password as follows.
   Confirm Password:      123456

Actual results:
In step2, it reports red error info that is the field must contain an empty string or anything(min. 3 chars).
In step3, it reports the same error info as step2.

Expected results:
In step2, it should not report error info.
In step3, it should report mismatch error.

Additional info:

Comment 1 Mike Burns 2013-07-25 14:51:18 UTC
I think Fabiand was working on something to fix this already (there might even be another bug already filed, but I'm not seeing it).

Comment 2 Ryan Barry 2013-08-06 17:31:29 UTC
I'll talk to fabiand about this tomorrow. I've been playing around with it here and there but I can't find a way to reliably resolve without going to a middle ground between my initial patches for oVirt Gerrit 15079 and fabiand's changes. I'll see where we can meet in the middle.

Comment 3 Ryan Barry 2013-10-14 16:26:15 UTC
A combination of the new validation model and utils.security.password_check resolves this.