fix multiple CVEs by updating brace-expansion to patched versions - #771
fix multiple CVEs by updating brace-expansion to patched versions#771sbouchet wants to merge 4 commits into
Conversation
|
Hi! I'm che-ai-assistant — I help with your pull requests. Available commands:
|
|
Pull Request images published ✨ Editor amd64: quay.io/che-incubator-pull-requests/che-code:pr-771-amd64 |
03977a9 to
dcf0f66
Compare
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (5)
🚧 Files skipped from review as they are similar to previous changes (4)
📝 WalkthroughWalkthroughThe pull request updates package override configurations across build, extension, test, and launcher manifests. It pins Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.rebase/CHANGELOG.md:
- Around line 11-24: Add launcher/package.json to the PR `#771` file list in the
`@sbouchet` changelog entry, preserving the existing entries and ordering.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 8414b664-d1f2-4507-b9ae-e5d4b80bf7ae
⛔ Files ignored due to path filters (19)
code/build/npm/gyp/package-lock.jsonis excluded by!**/package-lock.jsoncode/build/package-lock.jsonis excluded by!**/package-lock.jsoncode/extensions/che-api/package-lock.jsonis excluded by!**/package-lock.jsoncode/extensions/che-port/package-lock.jsonis excluded by!**/package-lock.jsoncode/extensions/che-remote/package-lock.jsonis excluded by!**/package-lock.jsoncode/extensions/che-resource-monitor/package-lock.jsonis excluded by!**/package-lock.jsoncode/extensions/copilot/chat-lib/package-lock.jsonis excluded by!**/package-lock.jsoncode/extensions/copilot/package-lock.jsonis excluded by!**/package-lock.jsoncode/extensions/css-language-features/package-lock.jsonis excluded by!**/package-lock.jsoncode/extensions/html-language-features/package-lock.jsonis excluded by!**/package-lock.jsoncode/extensions/json-language-features/package-lock.jsonis excluded by!**/package-lock.jsoncode/extensions/markdown-language-features/package-lock.jsonis excluded by!**/package-lock.jsoncode/extensions/npm/package-lock.jsonis excluded by!**/package-lock.jsoncode/package-lock.jsonis excluded by!**/package-lock.jsoncode/remote/package-lock.jsonis excluded by!**/package-lock.jsoncode/test/automation/package-lock.jsonis excluded by!**/package-lock.jsoncode/test/integration/browser/package-lock.jsonis excluded by!**/package-lock.jsoncode/test/sanity/package-lock.jsonis excluded by!**/package-lock.jsonlauncher/package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (33)
.rebase/CHANGELOG.md.rebase/add/code/build/npm/gyp/package.json.rebase/add/code/build/package.json.rebase/add/code/extensions/copilot/chat-lib/package.json.rebase/add/code/extensions/copilot/package.json.rebase/add/code/extensions/css-language-features/package.json.rebase/add/code/extensions/html-language-features/package.json.rebase/add/code/extensions/json-language-features/package.json.rebase/add/code/extensions/markdown-language-features/package.json.rebase/add/code/extensions/npm/package.json.rebase/add/code/package.json.rebase/add/code/test/automation/package.json.rebase/add/code/test/integration/browser/package.json.rebase/add/code/test/sanity/package.jsoncode/build/npm/gyp/package.jsoncode/build/package.jsoncode/extensions/che-api/package.jsoncode/extensions/che-port/package.jsoncode/extensions/che-remote/package.jsoncode/extensions/che-resource-monitor/package.jsoncode/extensions/copilot/chat-lib/package.jsoncode/extensions/copilot/package.jsoncode/extensions/css-language-features/package.jsoncode/extensions/html-language-features/package.jsoncode/extensions/json-language-features/package.jsoncode/extensions/markdown-language-features/package.jsoncode/extensions/npm/package.jsoncode/package.jsoncode/test/automation/package.jsoncode/test/integration/browser/package.jsoncode/test/sanity/package.jsonlauncher/package.jsonrebase.sh
|
Pull Request images published ✨ Editor amd64: quay.io/che-incubator-pull-requests/che-code:pr-771-amd64 |
RomanNikitenko
left a comment
There was a problem hiding this comment.
@sbouchet
I've got git changes when I built che-code:
che-code dockerfiles use v24.18.0 Node.js
could you try to rebuild che-code using v24.18.0 vestion?
RomanNikitenko
left a comment
There was a problem hiding this comment.
other looks good to me
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Stephane Bouchet <sbouchet@redhat.com>
Extends brace-expansion overrides to all affected packages to fix DoS via unbounded intermediate arrays (CRW-12223). Signed-off-by: Stephane Bouchet <sbouchet@redhat.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Stephane Bouchet <sbouchet@redhat.com>
dcf0f66 to
198d112
Compare
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
|
Pull Request images published ✨ Editor amd64: quay.io/che-incubator-pull-requests/che-code:pr-771-amd64 |
|
Pull Request images published ✨ Editor amd64: quay.io/che-incubator-pull-requests/che-code:pr-771-amd64 |
What does this PR do?
This PR fixes CVE-2026-13149 and CVE-2026-69152
brace-expansionversions are updated to5.0.9,2.1.4and1.1.18What issues does this PR fix?
https://redhat.atlassian.net/browse/CRW-11666
https://redhat.atlassian.net/browse/CRW-12223
How to test this PR?
Does this PR contain changes that override default upstream Code-OSS behavior?
git rebasewere added to the .rebase folderSummary by CodeRabbit
Bug Fixes
Documentation
Chores