chore(deps): bump postcss to 8.5.10 (XSS via unescaped </style>)#417
chore(deps): bump postcss to 8.5.10 (XSS via unescaped </style>)#417calvinbrewer merged 1 commit intomainfrom
Conversation
Patches alert #102: PostCSS XSS via unescaped `</style>` in CSS stringify output. Vulnerable: < 8.5.10. Two postcss versions were in the lockfile: - 8.4.31 (next-pinned hard dependency, vulnerable) - 8.5.6 (transitive via tsup, vite peer, postcss-load-config — vulnerable) Both bumped to 8.5.10 (minimum patched), and the two entries collapse into a single deduped resolution since they have identical deps (nanoid, picocolors, source-map-js). Override added: ">=8.5.10". Note on the next-pinned 8.4.31: Next.js declares postcss exactly, not via a range. Override forces 8.5.10 in the lockfile despite the exact pin; postcss 8.5.x is API-compatible with 8.4.x (8.5.0 added container queries support, no breaks). Surgical lockfile edit covers ~16 references (peer-hash strings in tsup/vite/postcss-load-config snapshot keys, plus the package def + integrity and the snapshot leaf).
|
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
✨ 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 |
Summary
Lockfile state
Two postcss versions were present, both vulnerable:
Both replaced with `postcss@8.5.10`. Since the two entries have identical deps (`nanoid`, `picocolors`, `source-map-js`), they deduplicate into a single resolution.
On overriding next's exact pin
Next.js declares `postcss: '8.4.31'` as an exact dep (not a range). The root override `"postcss": ">=8.5.10"` forces 8.5.10 in the lockfile despite the exact pin. postcss 8.5.x is API-compatible with 8.4.x (8.5.0 added container-query support; no breaks), and Next.js's postcss usage is internal to its build pipeline.
Changes
Test plan