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

7 permissions "RowLevelSecurityFiltersModelView" don't exist but documentation mentions them #32205

Open
2 of 3 tasks
xavier-GitHub76 opened this issue Feb 10, 2025 · 1 comment
Labels
authentication:row-level-security Related to Row Level Security

Comments

@xavier-GitHub76
Copy link
Contributor

Bug description

7 permissions "RowLevelSecurityFiltersModelView" don't exist in Superset 4.0.2
but documentation mentions them.

https://github.com/apache/superset/blob/master/RESOURCES/STANDARD_ROLES.md

Image

Screenshots/recordings

No response

Superset version

4.0.2

Python version

3.9

Node version

16

Browser

Chrome

Additional context

No response

Checklist

  • I have searched Superset docs and Slack and didn't find a solution to my problem.
  • I have searched the GitHub issue tracker and didn't find a similar bug report.
  • I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.
Copy link

dosubot bot commented Feb 10, 2025

The discrepancy you're observing between the documentation and the actual implementation of permissions for RowLevelSecurityFiltersModelView in Superset 4.0.2 might be due to a few reasons. The RowLevelSecurityFiltersModelView is implemented as RLSRestApi in the codebase, and it includes permissions for CRUD operations and other functionalities [1]. However, the permissions matrix in the STANDARD_ROLES.md file might not have been updated to reflect the current state of the codebase.

Additionally, the RowLevelSecurityFiltersModelView is listed under ADMIN_ONLY_VIEW_MENUS, indicating that its permissions are intended for admin users only [2]. This could explain why certain permissions are not available to other roles as mentioned in the documentation.

If the permissions are indeed missing, it could be an oversight in the documentation or a feature that hasn't been implemented yet. It might be helpful to check the test suite for RowLevelSecurityFiltersModelView to verify the expected behavior and permissions [3].

To continue talking to Dosu, mention @dosu.


Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Other

@dosubot dosubot bot added the authentication:row-level-security Related to Row Level Security label Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
authentication:row-level-security Related to Row Level Security
Projects
None yet
Development

No branches or pull requests

1 participant