New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(manager/gitlabci): clean more than one slash #17132
Conversation
https://github.com/renovatebot/renovate/blob/main/lib/util/url.ts#L30 should probably remove also multiple leading slashes |
@viceice don't think this will work since that only trims 1 slash |
Change this in the same pr ? 🤔 |
sure, it's a very small change and probably expected. If it fails too many tests then we should do in seperate PR |
Head branch was pushed to by a user without write access
🎉 This PR is included in version 32.159.1 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes
Clear more than one slash from the include block in gitlabci.
Context
According to https://gitlab.com/gitlab-org/gitlab/-/issues/28987#note_392419953 there can be more than one slash.
Currently renovate is giving "Preventing access to file outside the base directory" error.
If we take care of extra slashes we are able to detect the dependencies without and error :
Ran on: https://gitlab.com/hasan.awad/myfirsttestdata
Closes #16600
Documentation (please check one with an [x])
How I've tested my work (please tick one)
I have verified these changes via: