| Issue 3400: | Reviewers plugin UI is broken due to mis-generated links in Gerrit 2.11 | |
| 4 people starred this issue and may be notified of changes. | Back to list |
***************************************************************** ***** ***** ***** !!!! THIS BUG TRACKER IS FOR GERRIT CODE REVIEW !!!! ***** ***** ***** ***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, CYANOGENMOD, ***** ***** INTERNAL ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.***** ***** ***** ***** THOSE ISSUES BELONG IN DIFFERENT ISSUE TRACKERS ***** ***** ***** ***************************************************************** Affected Version: Gerrit 2.11 with reviewers plugin from stable-2.11 branch What steps will reproduce the problem? 1. Create a project with "/" in name, e.g. "aa/bb" 2. Try clicking on Reviewers tab in project screen In the reverse proxy config "AllowEncodedSlashes On" is already configured. What is the expected output? What do you see instead? The reviewers plugin UI should be opened, but instead you get: "Code Review - Error The page you requested was not found, or you do not have permission to view this page." Please provide any additional information below. The generated link for Reviewers plugin web UI is: https://<Gerrit Instance URL>/#/x/reviewers/aa%2Fbb instead of https://<Gerrit Instance URL>/#/x/reviewers/aa/Fbb This used to work correctly in Gerrit 2.10. It looks like this is a regression in Gerrit 2.11 in the way plugins contribute links in project context with project names including "/".
May 31, 2015
Project Member
#1
David.Os...@gmail.com
Cc:
edwin.ke...@sap.com
Jun 1, 2015
@Edwin, this regression was caused by this commit: [1]. Can you have a look? * [1] https://github.com/gerrit-review/gerrit/commit/f2d4c991f675805acf6011481df6707171e907c3
Status:
AwaitingInformation
Jun 2, 2015
It is correct that the project name is encoded in the URL https://<Gerrit Instance URL>/#/x/reviewers/aa%2Fbb The Reviewers plugin should decode it.
Jun 2, 2015
The fix was moved to stable-2.11: https://gerrit-review.googlesource.com/68281
Jun 2, 2015
fixed in stable-2.11 branch
Status:
Submitted
|
|
| ► Sign in to add a comment |