-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
golang-crossbuild: fix the issue with the CVE-2022-24765 #34241
golang-crossbuild: fix the issue with the CVE-2022-24765 #34241
Conversation
This reverts commit b32084d.
hence this should help to avoid issues
This pull request doesn't have a |
/packaging |
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
💔 Build Failed
Expand to view the summary
Build stats
Test stats 🧪
Steps errorsExpand to view the steps failures
|
#34254 is the one actually changing the git correctly! Closing this one now |
💔 Build Failed
Expand to view the summary
Build stats
Test stats 🧪
Steps errorsExpand to view the steps failures
|
What does this PR do?
Fix the issue with the
CVE-2022-24765
, so the workspace is set as safeWhy is it important?
Otherwise, it will fail with:
How to test this PR locally
##. Further details
b32084d didn't work since the docker container requires the docker use so it can run the
installDependencies
function defined in `dev-tools/mage/pkgdeps.goSo I ended up using the suggestion from elastic/golang-crossbuild#232
Related issues