URL redirecting doesn't work |
|||||||||||
Issue descriptionFrom an email: "Stephen, looks like a login redirect issue? The link on the sheriffing page is https://sheriff-o-matic.appspot.com/chromium.perf, but I just clicked it and I get a login page and then it redirects and takes the "chromium.perf" part off my query. " I've experienced this a couple times. We should definitely fix this; if we put a link to SOM on a sheriffing doc page, and it doesn't redirect correctly, and they're unfamiliar with the tool, they could end up sheriffing the wrong tree.
,
Nov 10 2016
,
Nov 10 2016
This should be a simple fix with big impact now that there's multiple rotations. Can we get someone on this?
,
Nov 10 2016
I'll look at this today.
,
Nov 10 2016
assigning to martiniss@ as he intends to look into it
,
Dec 2 2016
Working on this. Small fix!
,
Dec 2 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/fb6cd3035cc307a93948a87be99320e24ece75c0 commit fb6cd3035cc307a93948a87be99320e24ece75c0 Author: Stephen Martinis <martiniss@chromium.org> Date: Fri Dec 02 22:34:45 2016 SOM: redirect to original page when asked to login BUG= 662283 Change-Id: I79965de74b387f6b78be96558be4726b9e4743f4 Reviewed-on: https://chromium-review.googlesource.com/416372 Reviewed-by: Sean McCullough <seanmccullough@chromium.org> Commit-Queue: Stephen Martinis <martiniss@chromium.org> [modify] https://crrev.com/fb6cd3035cc307a93948a87be99320e24ece75c0/go/src/infra/appengine/sheriff-o-matic/som/main.go
,
Dec 9 2016
,
Mar 4 2017
,
Apr 17 2017
,
May 30 2017
,
Aug 1 2017
,
Oct 14 2017
|
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by sullivan@chromium.org
, Nov 10 2016