deps(frontend): bump immutable, postcss, fast-uri and rspack [security] - #8156
Conversation
Clears five High advisories reported against the flagsmith-frontend image: - immutable 5.1.6 -> 5.1.9 (CVE-2026-59880, CVE-2026-59879) - postcss 8.5.15 -> 8.5.25 (GHSA-r28c-9q8g-f849) - fast-uri 3.1.2 -> 3.1.4 (CVE-2026-16221, CVE-2026-13676) - @rspack/* 2.0.5 -> 2.1.7 (CVE-2026-55603, via the http-proxy-middleware copy vendored under @rspack/core/compiled) Also picks up body-parser 1.20.6 and brace-expansion 2.1.3/5.0.8. Lockfile only: every range in package.json already permitted the patched versions, so no manifest or semver changes are needed.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
Docker builds report
|
✅ private-cloud · depot-ubuntu-latest-arm-16 — run #18854 (attempt 1)Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
🗂️ Previous results✅ private-cloud · depot-ubuntu-latest-16 — run #18854 (attempt 1)Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Skipped testsfirefox › tests/onboarding-tests.pw.ts › Onboarding › New user connects via the single-page onboarding flow @oss ✅ oss · depot-ubuntu-latest-16 — run #18854 (attempt 1)Playwright Test Results (oss - depot-ubuntu-latest-16)Details
✅ oss · depot-ubuntu-latest-arm-16 — run #18854 (attempt 1)Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
|
Visual Regression19 screenshots compared. See report for details. |
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature.Changes
Clears five High advisories flagged against the
flagsmith-frontendimage on 2.256.0 and 2.257.0. All five were already open Trivy alerts onmain—trivy.yamlsetsexit-code: 0, so they never blocked a build.Lockfile only: every range in
package.jsonalready permitted the patched versions, so no manifest changes are needed.Map/Setdegrade insert/lookup to a CPU DoS when an attacker controls keys.immutableis a transitive dep ofsassonly, used to compile SCSS at build time; no app code imports it.immutable5.1.9Listindices between 2^30 and 2^31 cause infinite loops, unbounded allocation or silent wraparound.immutable5.1.9sourceMappingURLcomments are followed without sandboxing.., disclosing arbitrary.mapfiles.postcsscomes in viacss-loader; it only ever processes our own CSS duringnpm run bundle, never untrusted input at runtime.postcss8.5.25fast-uricomes in viaajv, used by rspack/webpack/terser to validate their own config schemas — never for host decisions.fast-uri3.1.4fast-uri3.1.4fixRequestBodylets an attacker injectmultipart/form-datafields, desynchronising the proxy from the backend.@rspack/core/compiled/http-proxy-middleware@4.0.0, invisible tonpm audit. Dev-server only; the runtime image never loads it.@rspack/*2.1.7, vendoring 4.2.0Also picks up
body-parser1.20.6 (GHSA-v422-hmwv-36x6) andbrace-expansion2.1.3/5.0.8 (GHSA-3jxr-9vmj-r5cp).So the exposure is nil in a deployed container — all six are build tooling that ships inside the image because
oss-frontendcopies the whole/build/frontend/(includingnode_modules) and the build deps live independenciesrather thandevDependencies. Onlyflagsmith-frontendis affected; the API and unified images carry the compiled bundle only, no npm packages. Splitting build from runtime deps so the toolchain stops shipping is a follow-up.Known remaining High in the production tree:
brace-expansionviaexpress-handlebars@6→glob→minimatch. That line has no patch; clearing it needsexpress-handlebars@8, a breaking major, so it is left for a separate PR.How did you test this code?
npm audit --omit=devno longer reports any of the six; the only High left is theexpress-handlebarschain noted above.@rspack/core@2.0.5vendorscompiled/http-proxy-middleware@4.0.0, and 2.1.7 vendors 4.2.0.ENV=selfhosted npm run bundlecompiles clean on rspack 2.1.7 — 0 errors, 24 pre-existing sass deprecation warnings.npm run test:unit— 25 suites, 284 tests, all passing.git diff frontend/package.jsonis empty, confirming the bumps sit inside the declared ranges.