Page MenuHomePhabricator

Wrong redirect when logging into grafana-rw from a grafana.w.o dashboard
Closed, ResolvedPublic

Description

We've recently introduced (in Iaf77445ba ) the ability to "log in" from a grafana.w.o dashboard and get redirected into grafana-rw.w.o.

However this mechanism doesn't always seem to work properly, as experienced and reported by @aborrero. Namely if idp/cas has to renew/refresh the session then the user isn't redirected to grafana-rw but to grafana.w.o with an extra ticket in the query string!

Example:

At this point if idp/cas don't need a new ticket all is well. However if there is indeed a need for a new ticket then:

Event Timeline

Change 640436 had a related patch set uploaded (by Filippo Giunchedi; owner: Filippo Giunchedi):
[operations/puppet@production] Revert "profile: redirect to grafana-rw with referer"

https://gerrit.wikimedia.org/r/640436

Change 640436 merged by Filippo Giunchedi:
[operations/puppet@production] Revert "profile: redirect to grafana-rw with referer"

https://gerrit.wikimedia.org/r/640436

jijiki triaged this task as Medium priority.Nov 10 2020, 3:58 PM

Change 641150 had a related patch set uploaded (by Filippo Giunchedi; owner: Filippo Giunchedi):
[operations/puppet@production] grafana: don't redirect cas (re)-auth requests

https://gerrit.wikimedia.org/r/641150

Change 641150 merged by Filippo Giunchedi:
[operations/puppet@production] grafana: don't redirect cas (re)-auth requests

https://gerrit.wikimedia.org/r/641150

Change 641164 had a related patch set uploaded (by Filippo Giunchedi; owner: Filippo Giunchedi):
[operations/puppet@production] profile: redirect to grafana-rw with referer

https://gerrit.wikimedia.org/r/641164

Change 641164 merged by Filippo Giunchedi:
[operations/puppet@production] profile: redirect to grafana-rw with referer

https://gerrit.wikimedia.org/r/641164

fgiunchedi claimed this task.

This is fixed now \o/