Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dashboard RBAC Favourite issue #19849

Closed
stevensuting opened this issue Apr 26, 2022 · 2 comments · Fixed by #24991
Closed

Dashboard RBAC Favourite issue #19849

stevensuting opened this issue Apr 26, 2022 · 2 comments · Fixed by #24991
Assignees
Labels
#bug Bug report validation:validated A committer has validated / submitted the issue or it was reported by multiple users

Comments

@stevensuting
Copy link

If a dashboard is assigned to a role, then user under that role makes that dashboard a favourite by clicking on the start, then if the role is removed from the dashboard, the user can still see the dashboard, but will not be able to access the content of the dashboard.

How to reproduce the bug

  1. Create a dashboard (example_dash) and assign a role (example_role) to it via the RBAC option via an ADMIN user
  2. Login with a non admin user who has the example_role
  3. Make the example_dash a favourite dashboard, logout
  4. Login in with Admin user and remove the example_role from the example_dash5.
  5. Login with the non admin user and you will see that example_dash is still visible.

Expected results

Dashboard Board should not be visible if the role has been removed from Dashboard RBAC irrespective of the favourite status.

Environment

(please complete the following information):

  • browser type and version: Chrome
  • superset version: 1.4.2
  • python version: 3,8
  • any feature flags active: RBAC
@stevensuting stevensuting added the #bug Bug report label Apr 26, 2022
@rusackas rusackas added the validation:required A committer should validate the issue label Jun 1, 2023
@rusackas
Copy link
Member

rusackas commented Jun 1, 2023

This issue and the superset version reported are old enough that I was tempted to close it, but it sounds... worrisome.

@stevensuting @jinghua-qa @sadpandajoe are any of you able to validate this problem still exists?

@pandinug
Copy link

pandinug commented Jun 1, 2023

Can confirm that this is very much still a thing on 2.1.0.
Easy to reproduce by creating a random role. Next, add the role to a test user, dataset and a dashboard.
Now log in with the test user and see that the dashboard is there. Be happy, star the dashboard. Log out again, or not.
Next, the admin takes away your role, possibly because you are switching to another department.
You log in to Superset again and your dashboard is still there. Not only that, also all the datasets are accessibly via the charts on the dashboard.

@rusackas rusackas added validation:validated A committer has validated / submitted the issue or it was reported by multiple users and removed validation:required A committer should validate the issue labels Jul 26, 2023
@yousoph yousoph removed their assignment Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#bug Bug report validation:validated A committer has validated / submitted the issue or it was reported by multiple users
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants