chore(deps): patch cli-common, backend-plugin-api, techdocs-node#4411
chore(deps): patch cli-common, backend-plugin-api, techdocs-node#4411alizard0 wants to merge 4 commits intoredhat-developer:release-1.8from
Conversation
JessicaJHee
left a comment
There was a problem hiding this comment.
Looks good! Just confirming that my understanding is correct - these 2 versions are left alone because they're safe?
├─ @backstage/backend-plugin-api@npm:1.7.0
│ └─ @backstage/cli-common@npm:0.1.18 (via npm:^0.1.18)
- CVE fixed in 0.1.17
├─ @backstage/plugin-permission-node@npm:0.10.10
│ └─ @backstage/backend-plugin-api@npm:1.7.0 (via npm:^1.7.0)
- CVE fixed in 1.6.1
package.json
Outdated
| "@backstage/plugin-scaffolder-node@^0.8.2": "patch:@backstage/plugin-scaffolder-node@npm%3A0.11.0#./.yarn/patches/@backstage-plugin-scaffolder-node-npm-0.11.0-2e81f51535.patch", | ||
| "@backstage/cli-common@^0.1.15": "patch:@backstage/cli-common@npm%3A0.1.15#./.yarn/patches/@backstage-cli-common-npm-0.1.15-7e1948f753.patch", | ||
| "@backstage/backend-plugin-api@^1.4.2": "patch:@backstage/backend-plugin-api@npm%3A1.4.2#./.yarn/patches/@backstage-backend-plugin-api-npm-1.4.2-00e99e5ef3.patch", | ||
| "@backstage/backend-plugin-api@^1.4.3": "patch:@backstage/backend-plugin-api@npm%3A1.4.2#./.yarn/patches/@backstage-backend-plugin-api-npm-1.4.2-00e99e5ef3.patch" |
There was a problem hiding this comment.
you removed the 1.4.3 patch. I believe most deps were on 1.4.3 so we discussed keeping the 1.4.3 and resolving 1.4.2 versions to 1.4.3
There was a problem hiding this comment.
I re-imported the patch and added
"@backstage/backend-plugin-api": "patch:@backstage/backend-plugin-api@npm%3A1.4.3#./.yarn/patches/@backstage-backend-plugin-api-npm-1.4.3-b7f08217ef.patch"
in the yarn only exists 1.4.3 and its patch
|
The container image build workflow finished with status: |
|
@alizard0: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
the current PR need changes from #4424 otherwise rhdh won't start. issues with zod dependencies that might be fixed with the resolutions from that PR |
|
PR needs rebase. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Description
@backstage/plugin-techdocs-nodevulnerable to arbitrary code execution via MkDocs hooks.Upstream fixed it at backstage/backstage@66e08b0#diff-a00c2d14ca2579883df4436744e43f3ef63a192c850c36603bdaac92364354fcR3
There are three patches:
@backstage/plugin-techdocs-node- the one that had the vulnerability@backstage/cli-common- techdocs-node uses a method from this packageisChildPaththat required an update too.@backstage/backend-plugin-apiis another package that usesisChildPathand as this was patched for fixing the CVE, this package needed an update too.All patches were applied as resolutions in
package.jsonanddynamic-plugins/package.json, with an exception of@backstage-plugin-techdocsbeing applied only ondynamic-plugins/package.jsonbecause it is not used by package.jsonWhich issue(s) does this PR fix
PR acceptance criteria
Please make sure that the following steps are complete:
How to test changes / Special notes to the reviewer