Red Hat Bugzilla – Bug 1370570
C&U - WEB UI crashes when moving from calendar to daily/hourly selection
Last modified: 2016-11-30 14:53:09 EST
Yaacov, should this be moved to POST?
Backported: https://github.com/ManageIQ/manageiq/pull/10458
Assigning the BZ to Einat for verification.
Assigned. This still occurs in 5.6.2. Same steps to reproduce.
https://github.com/ManageIQ/manageiq/pull/11506
New commit detected on ManageIQ/manageiq/darga: https://github.com/ManageIQ/manageiq/commit/3959bbc655f180d29e89954a490cbe63309ffbd1 commit 3959bbc655f180d29e89954a490cbe63309ffbd1 Author: yaacov <kobi.zamir@gmail.com> AuthorDate: Sun Sep 25 18:29:58 2016 +0300 Commit: yaacov <kobi.zamir@gmail.com> CommitDate: Tue Sep 27 16:23:16 2016 +0300 C&U - WEB UI crashes when moving from calendar to daily/hourly selection It is posible to send an empty string as a date from the UI, we currently only protect against nil date but not against an empty string. In this PR we add a check for empty string date. PR #10458 did not reset the date in cases where the date was empty, it only protected the sanity checks. This PR also reset the date in cases where the date string is mepty. Bugzilla https://bugzilla.redhat.com/show_bug.cgi?id=1370570 .../application_controller/performance.rb | 4 +- .../application_controller/performance_spec.rb | 50 ++++++++++++++++++++++ 2 files changed, 52 insertions(+), 2 deletions(-)
Merged upstream: https://github.com/ManageIQ/manageiq/pull/11506
Verified. UI no longer crashes.
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://rhn.redhat.com/errata/RHSA-2016-2839.html