Bug 1290350 - [User portal] No error message while leaving 'username:' and 'password:' fields blank
Summary: [User portal] No error message while leaving 'username:' and 'password:' fiel...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.UserPortal
Version: 3.6.1.1
Hardware: Unspecified
OS: Linux
high
medium
Target Milestone: ovirt-3.6.3
: 3.6.3
Assignee: Alexander Wels
QA Contact: Petr Matyáš
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-12-10 10:06 UTC by Krishna Babu K
Modified: 2016-02-18 11:07 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-02-18 11:07:39 UTC
oVirt Team: UX
Embargoed:
rule-engine: ovirt-3.6.z+
mgoldboi: planning_ack+
ecohen: devel_ack+
rule-engine: testing_ack+


Attachments (Terms of Use)
screen shot to refer the 'both the fields empty' scenario (236.66 KB, image/png)
2015-12-10 10:06 UTC, Krishna Babu K
no flags Details
screen shot to refer the 'one field empty' scenario (237.22 KB, image/png)
2015-12-10 10:08 UTC, Krishna Babu K
no flags Details
screenshot (60.25 KB, image/png)
2016-01-21 11:18 UTC, Petr Matyáš
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 51816 0 ovirt-engine-3.6 MERGED userportal: show fill in fields message 2016-01-18 15:55:00 UTC
oVirt gerrit 52564 0 ovirt-engine-3.6.2 MERGED userportal: show fill in fields message 2016-01-24 09:26:20 UTC

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


Note You need to log in before you can comment on or make changes to this bug.