[Predator-Fracas] Regression range and feedback links in Fracas broke |
||
Issue description[Predator-Fracas] Regression range and feedback links in Fracas broke Example: In https://fracas.googleplex.com/workspace/b32fb3d3-fc95-489a-b5c8-005cdc5490da The regression range link is null The feedback link is https://fracas.googleplex.com/findit-for-me.googleplex.com/crash/fracas-result-feedback?key=ahpzfmdvb2dsZS5jb206ZmluZGl0LWZvci1tZXJBCxITRnJhY2FzQ3Jhc2hBbmFseXNpcyIoNzk5NjIzNTY4MWJlM2I5NjY0YjkwNjNlZWJhZWI4YjNmZjBiNDFjNgw
,
Dec 3 2016
OK, I have a patch for the regression range link. For the feedback link, would you please update the message Findit sends to Fracas to include "https://"? That's what's causing confusion on the Fracas side, and while I can just prepend it here, that's hackier than it needs to be. Thanks!
,
Dec 5 2016
Sure, thanks.
,
Dec 5 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/7c4f1005adb1a008de0e4abc380815b0831b9eb2 commit 7c4f1005adb1a008de0e4abc380815b0831b9eb2 Author: katesonia <katesonia@chromium.org> Date: Mon Dec 05 19:21:00 2016 [Predator] Add https:// in front of feedback link. As requested in bug 670405 . BUG= 670405 TBR=stgao@chromium.org, wrengr@chromium.org Review-Url: https://codereview.chromium.org/2556463002 [modify] https://crrev.com/7c4f1005adb1a008de0e4abc380815b0831b9eb2/appengine/findit/crash/findit_for_chromecrash.py [modify] https://crrev.com/7c4f1005adb1a008de0e4abc380815b0831b9eb2/appengine/findit/handlers/crash/test/crash_handler_test.py
,
Dec 5 2016
Thanks! Fracas-side change at https://team-review.git.corp.google.com/#/c/56445/. |
||
►
Sign in to add a comment |
||
Comment 1 by kateso...@chromium.org
, Dec 1 2016Status: Assigned (was: Available)