Bug 1290350

Summary: [User portal] No error message while leaving 'username:' and 'password:' fields blank
Product: [oVirt] ovirt-engine Reporter: Krishna Babu K <kkrothap>
Component: Frontend.UserPortalAssignee: Alexander Wels <awels>
Status: CLOSED CURRENTRELEASE QA Contact: Petr Matyáš <pmatyas>
Severity: medium Docs Contact:
Priority: high    
Version: 3.6.1.1CC: amarchuk, bugs, mgoldboi, oourfali, pmatyas, sbonazzo, ykaul
Target Milestone: ovirt-3.6.3Flags: rule-engine: ovirt-3.6.z+
mgoldboi: planning_ack+
ecohen: devel_ack+
rule-engine: testing_ack+
Target Release: 3.6.3   
Hardware: Unspecified   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-02-18 11:07:39 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: UX RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
screen shot to refer the 'both the fields empty' scenario
none
screen shot to refer the 'one field empty' scenario
none
screenshot none

Description Krishna Babu K 2015-12-10 10:06:29 UTC
Created attachment 1104292 [details]
screen shot to refer the 'both the fields empty' scenario

Description of problem:

At userportal login page if user do not provide any credentials in 'username:' and 'password:' fields no error message is showing up, just an empty red square box pops up with error sign in it. It is applicable even when we leave one of those fields empty.

Version-Release number of selected component (if applicable):

3.6.1.1-0.1.el6

How reproducible:

Always

Steps to Reproduce:
1. Go to userportal login page.
2. Leave both 'username:' and 'password:' fields or one of them empty.
3. Click on login.

Actual results:

No error message

Expected results:

error message should be present, some thing like "Please fill in all fields"

Additional info:
screenshot attached for reference.

Comment 1 Krishna Babu K 2015-12-10 10:08:30 UTC
Created attachment 1104293 [details]
screen shot to refer the 'one field empty' scenario

Comment 2 Einav Cohen 2015-12-15 13:33:57 UTC
we have already fixed the issue for empty password (see bug 1254940 and bug 1251003), however this is probably fixed only for non-empty user-name.
need to make sure that the solution implemented for bug 1254940 and bug 1251003 applies for an 'empty-user-name and empty-password' situation as well. 

(consult with Greg, assignee of bug 1254940 and bug 1251003, if necessary)

Comment 3 Einav Cohen 2015-12-15 18:39:36 UTC
(In reply to Einav Cohen from comment #2)
> this is probably fixed only for non-empty user-name.

another possibility is that this issue was fixed somehow only for the web-admin login page and not the user-portal login page. 

unrelated note: in 4.0, there are no user portal login page and web-admin login page - only SSO login page. 

So for 4.0 - please ensure that an appropriate error message appears for the SSO login page. 
for 3.6 - need a completely different patch that ensures that an appropriate error message appears for the web-admin and user-portal login pages.

Comment 4 Red Hat Bugzilla Rules Engine 2016-01-20 11:39:27 UTC
Bug tickets that are moved to testing must have target release set to make sure tester knows what to test. Please set the correct target release before moving to ON_QA.

Comment 5 Red Hat Bugzilla Rules Engine 2016-01-20 11:40:54 UTC
Bug tickets that are moved to testing must have target release set to make sure tester knows what to test. Please set the correct target release before moving to ON_QA.

Comment 6 Petr Matyáš 2016-01-21 11:18:31 UTC
Created attachment 1116899 [details]
screenshot

There still isn't any message, just red squares.

Tested on 3.6.2-10

Comment 7 Alexander Wels 2016-01-21 13:51:47 UTC
So it appears the patch didn't get backported to the 3.6.2 branch, just 3.6. I will backport it to 3.6.2

Comment 8 Red Hat Bugzilla Rules Engine 2016-01-21 23:03:45 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 9 Petr Matyáš 2016-01-29 12:10:27 UTC
Verified on 3.6.3-1