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

Improve wrong domain auth message #79

Closed
rafrombrc opened this issue May 16, 2017 · 4 comments
Closed

Improve wrong domain auth message #79

rafrombrc opened this issue May 16, 2017 · 4 comments
Assignees
Milestone

Comments

@rafrombrc
Copy link
Member

When a user is authenticated via Google using an account other than one that has access to sql.t.m.o, the error message is misleading ("sql.telemetry.mozilla.org is not redirecting properly", see https://bugzilla.mozilla.org/show_bug.cgi?id=1362348).

@rafrombrc rafrombrc added this to the 5 milestone Jun 13, 2017
@rafrombrc rafrombrc modified the milestones: 6, 5 Jun 21, 2017
@alison985
Copy link

When I first tried to re-create this last week it kept going back to the list of google accounts without showing an error message. In the weekly meeting it was suggested to try it in a different browser.

In firefox 53.0.3 (64-bit) today, staging give this message:
Screenshot 2017-06-22 13.39.17.png and landed on this URL: https://pipeline-sql.stage.mozaws.net/oauth/google_callback?state=/&code=4/8tloZ2_by3TLqrqeIGi-9i7D4rcZehDhfIosb7FvVbY#

In a Chrome incognito window today, it first gives a too many redirects error message before finalizing landing on a 500 internal server error at https://pipeline-sql.stage.mozaws.net/oauth/google_callback?state=/&code=4/XZuM-zib6vX0QVslLECcqptyZTmh8N3bxKrbjTU5ikU#

@alison985 alison985 self-assigned this Jun 22, 2017
@alison985
Copy link

You can also hit a 429 Too Many Requests page. "50 per 1 hour".

@alison985
Copy link

To be clear, to re-create this situation logout of re:Dash, logout of a mozilla google account, login to a personal or other google account, try to log into redash.

@alison985
Copy link

VictorC verified fixed via email on 7/31.

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

3 participants