Bug 1467146 - Unable to view Reports if a member has a custom Role - indefinite spinning wheel
Summary: Unable to view Reports if a member has a custom Role - indefinite spinning w...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.8.0
Hardware: Unspecified
OS: Unspecified
urgent
high
Target Milestone: GA
: 5.9.0
Assignee: Martin Hradil
QA Contact: Niyaz Akhtar Ansari
URL:
Whiteboard: spinner:rbac:report
Depends On:
Blocks: 1468336
TreeView+ depends on / blocked
 
Reported: 2017-07-03 04:35 UTC by tachoi
Modified: 2020-08-13 09:33 UTC (History)
9 users (show)

Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1468336 (view as bug list)
Environment:
Last Closed: 2018-03-06 14:35:45 UTC
Category: Bug
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
custom role (67.41 KB, image/png)
2017-07-03 04:35 UTC, tachoi
no flags Details
spinning wheel view (268.75 KB, image/png)
2017-07-03 04:46 UTC, tachoi
no flags Details

Description tachoi 2017-07-03 04:35:34 UTC
Created attachment 1293727 [details]
custom role

Description of problem:
Having different Roles for our customers. However when we give them view permission to Reports, they are just getting a loading with "Spinning wheel" indefinitely.

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

How reproducible:

Steps to Reproduce:
1.Create Custom Role with attached screenshot permission list
2.Create Custom group with assigning this Custom Role
3.Create new user with assigning this custom group
4.Login with new created user and navigate to built-in reports accordion

Actual results:
Only see spinning wheel indefinitely

Expected results:
Navigate Reports menu based on permission

Additional info:
Able to observe "ERROR -- : An unauthorized connection attempt was rejected" from production.log

Comment 3 tachoi 2017-07-03 04:46:09 UTC
Created attachment 1293729 [details]
spinning wheel view

Comment 7 CFME Bot 2017-07-06 16:28:45 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/5d36640dacc79b2aa3ac3978ddc34de37d74575c

commit 5d36640dacc79b2aa3ac3978ddc34de37d74575c
Author:     Martin Hradil <mhradil>
AuthorDate: Thu Jul 6 15:37:08 2017 +0000
Commit:     Martin Hradil <mhradil>
CommitDate: Thu Jul 6 15:37:08 2017 +0000

    Don't x_node_set on a tree which doesn't exist
    
    Calling `x_node_set` when a tree doesn't exist works, but actually creates the entry that `tree_exists?` checks to see if such a tree exists.
    
    When the user doesn't have the right to see such a tree, that's problematic, since it breaks the assumption that `tree_exists?` returns true only for trees that were actually initialized.
    
    The leads to the calling `lock_tree` on a nonexistent tree, leading to a JS exceptions ("No element has been found").
    
    This makes us actually check if the tree already exists before calling `x_node_set` on it.
    
    (Would prefer to fix `x_node_set` instead, but.. this is a bugfix, and I really don't want to introduce more bugs. :))
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1467146

 app/controllers/report_controller.rb | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comment 10 Niyaz Akhtar Ansari 2017-11-06 09:03:37 UTC
Verified in Version 5.9.0.5


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