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

"Open in Iodide" button should not be available for queries accessing restricted datasets (e.g. search) #59

Closed
wlach opened this issue Oct 29, 2019 · 5 comments

Comments

@wlach
Copy link

wlach commented Oct 29, 2019

One mitigation item mentioned by the RRA we did on the Iodide server was that the "open in iodide" button should not be made available for queries that use a restricted dataset (e.g. search).

If this is relatively easy, we should probably just do it. /cc @jezdez @openjck

@jezdez
Copy link
Contributor

jezdez commented Oct 31, 2019

Hm, I don't think we can identify which of the data sources are restricted programmatically, so we'd have to maintain a list of them in redash-stmo instead and check it before doing the request against Iodide.

@wlach
Copy link
Author

wlach commented Oct 31, 2019

Hm, I don't think we can identify which of the data sources are restricted programmatically, so we'd have to maintain a list of them in redash-stmo instead and check it before doing the request against Iodide.

I think it would be best to hide (or disable) the button for these cases. But I don't think having a hard-coded list would be so bad -- maybe it's best to do it as a whitelist, rather than a blacklist (in case a new secret data source is added)

@jezdez
Copy link
Contributor

jezdez commented Nov 4, 2019

Sure, which data sources would be allowed?

@wlach
Copy link
Author

wlach commented Nov 4, 2019

Sure, which data sources would be allowed?

Maybe just the list that is assigned to the group "default"? Actually, now that I look at it, could we just filter on that somehow? It seems to contain most of the things people would reasonably want to make iodide dashboards out of:

https://sql.telemetry.mozilla.org/queries/65817/source

@openjck
Copy link
Contributor

openjck commented Nov 14, 2019

I've moved this issue to iodide-project/redash-iodide#1 since the Iodide plugin now lives there.

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

No branches or pull requests

3 participants