Loading
Say which relative URL roots the metrics path decode cannot fix
What does this MR do?
Adds the line suggested in the review of !9929 to the decode_relative_url comment: decoding the relative URL root restores only the characters Go escapes again when Prometheus builds the scrape URL, and Go escapes some characters a root holds as written. So a%2Cb is requested as a,b and my(gitlab) as my%28gitlab%29, which Puma answers with a 404. The Linux package cannot correct that, because Prometheus accepts no raw path.
Comment only: no behavior change.
Related issues
Follow-up to !9929 (merged). Related to #9933 (closed).
Checklist
See Definition of done.
For anything in this list which will not be completed, please provide a reason in the MR discussion.
Required
- MR title and description are up to date, accurate, and descriptive.
- MR targeting the appropriate branch.
- Latest Merge Result pipeline is green.
- When ready for review, MR is marked as ready (not a draft) and a reviewer is requested. Pick the reviewer suggested by the reviewer roulette, or one of the project maintainers.
For GitLab team members
If you don't have access to this, the reviewer should trigger these jobs for you during the review process.
- The manual
Trigger:ee-packagejobs have a green pipeline running against latest commit.- To debug QA failures, refer to the Investigate QA failures section.
- If
config/softwareorconfig/patchesdirectories are changed, make sure thebuild-package-on-all-osjob within theTrigger:ee-packagedownstream pipeline succeeded. - If you are changing anything SSL related, then the
Trigger:package:fipsmanual job within theTrigger:ee-packagedownstream pipeline must succeed. - If CI configuration is changed, the branch must be pushed to
dev.gitlab.orgto confirm regular branch builds aren't broken.
Expected (please provide an explanation if not completing)
- Test plan indicating conditions for success has been posted and passes. Not applicable: comment only.
- Documentation created/updated. Not applicable: comment only.
- Tests added. Not applicable: comment only.
- Integration tests added to GitLab QA. Not applicable: comment only.
- Equivalent MR/issue for the GitLab Chart opened. Not applicable: Linux package code comment.
- Validate potential values for new configuration settings. Formats such as integer
10, duration10s, URIscheme://user:passwd@host:portmay require quotation or other special handling when rendered in a template and written to a configuration file. Not applicable: no new setting.
Reviewer checklist
- As soon as you begin reviewing, select Start a review on your first comment so the MR shows Reviewer started review.