Move security patch from main project to `gitlab-web-ide`
MR:
- fix: Run security patch before pack (!401 - merged)
- Upgrade @gitlab/web-ide (gitlab!177628 - merged)
Description
To mitigate https://gitlab.com/gitlab-org/security/gitlab/-/issues/1092, we patched the @gitlab/web-ide package used in the main project to:
- Remove unnecessary
.htmlfiles - Replace usage of
searchParams...parentOrigin
See this file:
We want to remove this patch and handle this modification of the VSCode assets here.
Acceptance Criteria
-
Move the web_ide_assets_spectest into this project.- Maybe this is an "integration" test since we're not actually testing units... Maybe we should add a sibling to
packages/folder fortests/which house our tests that require the whole project to be built. - We'll need to make sure this test runs on CI.
- This has been moved to follow-up Move Web IDE assets specs from main project (#431 - closed)
- Maybe this is an "integration" test since we're not actually testing units... Maybe we should add a sibling to
-
Update packages/vscode_buildto apply the changes that would make the test pass
Edited by Paul Slaughter