Page MenuHomePhabricator

Clicking on entries in Gerrit's "show blame" view results in 404
Closed, ResolvedPublicBUG REPORT


To reproduce:

Looks like the generated URL really needs to be

(i.e., prefixed with /r/)

Event Timeline

dancy created this task.Jul 3 2020, 8:29 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 3 2020, 8:29 PM
QChris changed the subtype of this task from "Task" to "Bug Report".
dancy updated the task description. (Show Details)Jul 3 2020, 8:43 PM
Krinkle added a subscriber: Krinkle.EditedJul 3 2020, 8:49 PM

As a workaround (for users reading this) you can click on "Gitiles" (top left of diff) instead of "Show blame". Then click [blame]. The commit references there do open fine, and the Change-Ids are also clickable which take into the Gerrit context.

Change 609590 had a related patch set uploaded (by QChris; owner: Christian Aistleitner):
[operations/software/gerrit@deploy/wmf/stable-3.2] Bump gerrit.war to Gerrit 3.2.2-102-g3bbb138e13

Change 609590 merged by QChris:
[operations/software/gerrit@deploy/wmf/stable-3.2] Bump gerrit.war to Gerrit 3.2.2-102-g3bbb138e13

QChris added a subscriber: QChris.Jul 5 2020, 9:58 PM

Fix (Thanks @Paladox) is deployed and effective.

QChris closed this task as Resolved.Jul 5 2020, 9:58 PM
QChris claimed this task.