Skip to content

fix: upgrade postcss to 8.5.18 (GHSA-r28c-9q8g-f849) - #31317

Closed
anupamme wants to merge 1 commit into
ionic-team:mainfrom
anupamme:fix-repo-ionic-framework-ghsa-r28c-9q8g-f849-postcss
Closed

anupamme wants to merge 1 commit into
ionic-team:mainfrom
anupamme:fix-repo-ionic-framework-ghsa-r28c-9q8g-f849-postcss

Conversation

@anupamme

@anupamme anupamme commented Aug 2, 2026

Copy link
Copy Markdown

Summary

Upgrade postcss from 8.5.3 to 8.5.18 to fix GHSA-r28c-9q8g-f849.

Vulnerability

Field Value
ID GHSA-r28c-9q8g-f849
Severity HIGH
Scanner trivy
Rule GHSA-r28c-9q8g-f849
File packages/vue-router/package-lock.json (dependency: postcss)
Assessment Present in dependency tree, not confirmed reachable

Description: PostCSS: Path Traversal in Previous Source Map Auto-Loading (sourceMappingURL) leads to Arbitrary .map File Disclosure

Changes

  • packages/vue-router/package.json
  • packages/vue-router/package-lock.json

Behavior Preservation

The change is scoped to 2 files on the vulnerable path; it only tightens handling of untrusted input and leaves valid inputs unaffected.


This patch removes an exploit primitive — a code pattern that, while not independently exploitable today, could be chained with other weaknesses by automated exploit-development tooling. Proactive removal of such primitives raises the bar against increasingly capable automated attack tools.


Automated security fix by OrbisAI Security

Automated dependency upgrade by OrbisAI Security
@anupamme
anupamme requested a review from a team as a code owner August 2, 2026 01:11
@anupamme
anupamme requested a review from thetaPC August 2, 2026 01:11
@vercel

vercel Bot commented Aug 2, 2026

Copy link
Copy Markdown

@anupamme is attempting to deploy a commit to the Ionic Team on Vercel.

A member of the Team first needs to authorize it.

@anupamme

anupamme commented Sep 1, 2026

Copy link
Copy Markdown
Author

Flagging overlap with #31318, which also touches `packages/vue-router`'s postcss resolution.

This PR targets postcss 8.5.18 (fixes GHSA-r28c-9q8g-f849). However, `main` has since moved to postcss 8.5.14 by default (no override), and there's a further postcss fix in 8.5.23 for the case where `opts.from` is unset. 8.5.18 alone wouldn't cover that.

I've updated #31318 to rebase onto current `main` and pin the override to postcss 8.5.23 (chosen over the latest 8.5.26 since 8.5.24 reportedly introduced a BOM regression for some consumers), with before/after `npm ls postcss` verification in the description.

To avoid landing two overlapping postcss security PRs, would it make sense to consolidate on whichever of these two you'd rather merge? Happy to close #31318 if you'd prefer to bump the target here to 8.5.23 instead, or vice versa.

@thetaPC

thetaPC commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Thanks for the PR! This ended up being handled in #31436, so we're closing this one out. Appreciate you digging into it.

@thetaPC thetaPC closed this Sep 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: vue @ionic/vue package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants