Bug 1270855

Summary:	scoped_search - err 500 on searching on invalid field
Product:	Red Hat Satellite	Reporter:	Roman Plevka <rplevka>
Component:	WebUI	Assignee:	John Mitsch <jomitsch>
WebUI sub component:	Katello	QA Contact:	Roman Plevka <rplevka>
Status:	CLOSED ERRATA	Docs Contact:
Severity:	medium
Priority:	unspecified	CC:	bbuckingham, sthirugn
Version:	Nightly	Keywords:	Triaged
Target Milestone:	Unspecified
Target Release:	Unused
Hardware:	Unspecified
OS:	All
URL:	http://projects.theforeman.org/issues/12140
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-07-27 08:57:52 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:

Description Roman Plevka 2015-10-12 14:11:17 UTC

Description of problem:
Applies to all katello entities:
on sending a query which uses a non-existing field, server returns error 500, but the GUI does not handle this error.
This results in 'dead' search form.

Request url:
https://<fqdn>/katello/api/v2/content_views?nondefault=true&organization_id=3&page=1&search=foo+%3D+bar&sort_by=name&sort_order=ASC

Response:
displayMessage:"Field 'foo' not recognized for searching!"
errors:Array:
 0:"Field 'foo' not recognized for searching!"

Version-Release number of selected component (if applicable):
katello-2.4.0-6.nightly.el7.noarch
tfm-rubygem-scoped_search-3.2.2-2.el7.noarch

How reproducible:
every time on all katello entities

Steps to Reproduce:
1. go to a katello entity page (e.g. Content Views)
2. input the seaerch query with invalid field name - e.g. foo = bar
3. Hit search
4. No response on GUI (Error in the captured repsonse headers)

Actual results:
the error is not being handled by UI, renders the form unusable.

Expected results:
Notification is shown or some default behavior takes place.

Additional info:

Comment 1 Roman Plevka 2015-10-12 14:13:57 UTC

Created redmine issue http://projects.theforeman.org/issues/12140 from this bug

Comment 3 Bryan Kearney 2015-12-17 19:04:13 UTC

Moving to POST since upstream bug http://projects.theforeman.org/issues/12140 has been closed

Comment 4 John Mitsch 2016-02-22 16:15:31 UTC

Moving this back to NEW as the issue status was changed back (it is still a bug upstream) http://projects.theforeman.org/issues/12140

Comment 5 John Mitsch 2016-03-02 20:03:46 UTC

This was fixed upstream https://github.com/Katello/katello/pull/5819

Comment 6 Roman Plevka 2016-03-25 23:15:42 UTC

Ok, this is working pretty nice now, but it still generates the empty suggestion instead showing the error in the suggestion box just like Foreman does.

I'm putting this to FAILEDQA. If you think this should be reported as a standalone bug, let me know and I'll put this one to VERIFIED

Comment 8 John Mitsch 2016-03-28 12:48:15 UTC

(In reply to Roman Plevka from comment #6)
> Ok, this is working pretty nice now, but it still generates the empty
> suggestion instead showing the error in the suggestion box just like Foreman
> does.
> 
> I'm putting this to FAILEDQA. If you think this should be reported as a
> standalone bug, let me know and I'll put this one to VERIFIED

+1 to a standalone bug for the empty suggestion box, I actually have opened up an upstream bug here for this issue and can clone it to BZ. http://projects.theforeman.org/issues/13118

Comment 10 Roman Plevka 2016-04-11 08:13:43 UTC

VERIFIED

putting this bug to verified as the other issue now has its own BZ #1322088

Comment 12 errata-xmlrpc 2016-07-27 08:57:52 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2016:1500