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 refresh query result needs view_query permission #2597

Closed
loskultos opened this issue Jun 18, 2018 · 8 comments
Closed

Dashboard refresh query result needs view_query permission #2597

loskultos opened this issue Jun 18, 2018 · 8 comments
Labels

Comments

@loskultos
Copy link

Issue Summary

Dashboard widgets refresh functionality only works when the user has view_query permission. view_query permission allows the user to see the query source

Expected Behavior

When the user has execute_query permission the refresh in the dashboard widgets should work without the user be able to acutally see the query source - so shouldn't need the view_query permission.

Technical details:

  • Redash Version: 4.0.1+b4038
@guzhenping
Copy link

l cannot set time to refresh dashboard widgets. when l click '1 minute' button, it works. but if l refresh website(F5),the setting is from "1 minute" to "Refresh".

Does this bug cause ?

@Jeetah
Copy link

Jeetah commented Jun 27, 2018

We also need this fixed to make the new Redash available for our customers. It's not acceptable that they see the insides of our DB.

@leonceeo
Copy link

Also blocked by this issue, I'm preparing a demo for a potential client, and I need refresh without letting him see the queries.

@kocsmy
Copy link
Collaborator

kocsmy commented Jun 28, 2018

@guzhenping that's a different issue and it's pretty much how the dashboard refresh works at the moment. It's a "frontend" refresh so refreshing or restarting the browser will get this schedule lost. We'll change this behaviour in the future and this will be saved somehow.

@guzhenping
Copy link

@kocsmy get it ! thank you

@brainboog
Copy link

Is there a workaround for this bug? Refresh should be independent of the view_query permission, otherwise it's not possible to provide read-only dashboards for customers.

@thoechsmann
Copy link

thoechsmann commented Sep 26, 2018

We also need that.

Some people should be able to run a query with different parameters, but should not be able to change the sql query.

@arikfr
Copy link
Member

arikfr commented Jan 15, 2019

We've started a project to redo permissions in Redash, which will also address this use case. To follow along you can watch #3284.

@arikfr arikfr closed this as completed Jan 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

9 participants