Public role is too strong by default

[xavier-GitHub76]

Bug description

Hello,

to implement the 'public' role, the documentation (https://superset.apache.org/docs/security/#public) indicates that the PUBLIC_ROLE_LIKE variable must be updated and gives the example of using the GAMMA role.
It is also specified that the GAMMA role provides access for consultation.

By applying these recommendations and defining a “datasource access on” permission, an anonymous user can consult :

• a dashboard
• diagrams
• datasets
  (by accessing the various main menus)

Visible elements are related to the “datasource access on” permission but the gamma permissions used like template are too strong.

An anonymous user can start creating a diagram and freely query a dataset via the diagram editing screen.
It can also export all diagram data.
However, it cannot save the diagram.

In order to achieve “read only” behavior, you should set up a role limited to the strict minimum and use it as a reference for the “public” role.

Here are the permissions I've identified as mandatory for this role:

• can dashboard permalink on Superset
• can read on Chart
• can read on Dashboard
• can dashboard on Superset
• can explore json on Superset
• can read on DashboardPermalinkRestApi
• can write on DashboardPermalinkRestApi
• can time range on Api
• can csv on Superset

Of course, these permissions must be supplemented with the “datasource access on” permission.

With these permissions, an anonymous user will only be able to :

• access a dashboard via its permalink
• view the dashboard (and its diagrams)
• generate permalinks (on tabs, headers, dashboard and diagrams)
• filter a dashboard (including time ranges)
• export a dashboard (PDF or image)
• export a chart (CSV, Excel or image)

Best regards

How to reproduce the bug

1. Define a PUBLIC_ROLE_LIKE = GAMMA
2. On the "Public" role, add a “datasource access on” permission on a table
3. Visit superset like anonymous user
4. Click on "Dataset" menu
5. Click on public dataset
6. Edit a chart
7. Export data
8. Save chart (refused)

Screenshots/recordings

No response

Superset version

master / latest-dev

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.

[sfirke]

This is a good catch - I agree that Gamma permissions are too powerful for most people who want a Public viewer role.

The set of permissions I've given to people is:

can read on CssTemplate, can read on Chart, can read on Annotation, can read on Dashboard, can get on OpenApi, can write on DashboardFilterStateRestApi, can read on DashboardFilterStateRestApi, can write on DashboardPermalinkRestApi, can read on DashboardPermalinkRestApi, can write on ExploreFormDataRestApi, can read on ExploreFormDataRestApi, can write on ExplorePermalinkRestApi, can read on ExplorePermalinkRestApi, can list on FilterSets, can time range on Api, can query on Api, can query form data on Api, can filter on Superset, can slice json on Superset, can explore json on Superset, can validate sql json on Superset, can favstar on Superset, can dashboard permalink on Superset, can sql json on Superset, can queries on Superset, can csv on Superset, can dashboard on Superset, can slice on Superset, can annotation json on Superset, can share dashboard on Superset, can read on AdvancedDataType

I shared that list with someone on Slack and they said that they also had to add these: menu access on dashboards, can recent activity on logs, can userinfo on UserDBModelView

I would be interested in blending it with yours to get the narrowest set possible, then recommending that to people -- or simply adding it as a default role in Superset.

It's been a couple of years since I refined this list. I see a couple of mine that I think are clearly missing from yours, e.g., getting CSS templates and annotations. Are there are any on my list that seem problematic to you in terms of giving the user too much power?

[rusackas]

Maybe it's time to add a new built-in public role (Delta?) for this purpose, but just not wire it up to anything by default? Curious also to hear if/how @dpgaspar would like to weigh in on the above.

[Arcelone]

I agree with @rusackas. I struggle since approximately 3 weeks to configure a proper "Public" role mixed or not with the RBAC feature. A proper minimal Public role should be included or at least documented.

In addition I didn't found any documentation of permission. Is there one somewhere I have missed ? If not it's important to write one.

[Arcelone]

@xavier-GitHub76 what is the "can write to DashboardPermalinkRestApi" permission for?
I tried without and I didn't found what was blocked.

[sfirke]

@Arcelone I'm guessing it's so that dashboard permalinks can be accessed. E.g., accessing the dashboard at https://data.yourdomain.com/superset/dashboard/your-dashboard instead of https://data.yourdomain.com/superset/dashboard/101. Did you try that?

[Arcelone]

@sfirke I didn't get what you said.

When you share the dashboard permalink it looks like : https://data.yourdomain.com/superset/dashboard/aiiduzgxykrofbzh

The receiver paste it in his browser and here you go.

[sfirke]

Related discussion: #25938

[xavier-GitHub76]

I will precise each permission (and issues if missing)

[xavier-GitHub76]

Without "can write to DashboardPermalinkRestApi" : Issue #30004

[xavier-GitHub76]

Without "can time range on Api" : #30005

[xavier-GitHub76]

The display of dashboard is impossible (redirection to login page) without one of

• can dashboard on Superset
• can dashboard permalink on Superset
• can explore json on Superset
• can read on Chart
• can read on Dashboard
• can read on DashboardPermalinkRestApi

[xavier-GitHub76]

To download a chart as CSV/Excel/Image --> permission "can csv on Superset " (label not precise) #30317