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

Public URL for a dashboard that contains a text widget throws 500 server error #220

Closed
madalincm opened this issue Aug 24, 2017 · 4 comments
Assignees
Milestone

Comments

@madalincm
Copy link

Issue Summary:

Build identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:55.0) Gecko/20100101 Firefox/55.0

Steps to Reproduce:

  1. Open a dashboard that contains a text widget.
  2. Generate a public url and try to open that url.

Expected results:

The url can be loaded, the dashboard is displayed.

Actual results:

500 Error displayed in the console, the dashboard can't be accessed.

Notes/Issues:

Verified in FF55(Win7). Issue can be reproduced only in STMO stage and STMO prod

Example of a dashboard that contains a text widget: https://pipeline-sql.stage.mozaws.net/dashboard/textwidgetservererrorinpublicurl

Public URL that returns the server error: https://pipeline-sql.stage.mozaws.net/public/dashboards/fOO9BUW3svpUGgbqI59Honzgv5XjfufvsTllhqma?org_slug=default

@madalincm madalincm added the bug label Aug 24, 2017
@madalincm madalincm changed the title Public URL for a dashboard that contains a text widget throws 500 server errror Public URL for a dashboard that contains a text widget throws 500 server error Aug 24, 2017
@washort
Copy link

washort commented Aug 29, 2017

Looks like this is a result of the changes for #166: https://sentry.prod.mozaws.net/operations/data-stmo-stage/issues/642969/events/11758337/

@alison985 alison985 added this to the 10 milestone Sep 5, 2017
alison985 added a commit that referenced this issue Sep 11, 2017
Run queries with no cached result in public dashboards (re #220)
@madalincm
Copy link
Author

Verified fixed in FF55(Win7) in stmo-stage.

@washort
Copy link

washort commented May 16, 2019

We are dropping this fix from m21 because run_query_sync has been removed upstream. We believe this will have low impact because queries with no results are run when added to a dashboard, meaning that this error will only occur until the query completes. The long term solution is probably the one in getredash#2844 .

@washort washort reopened this May 16, 2019
@jezdez jezdez modified the milestones: 10, 21 May 16, 2019
@jezdez
Copy link

jezdez commented Aug 2, 2019

Closing as this has not been requested again.

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

No branches or pull requests

4 participants