Skip to content

deps: fix webpack SSRF alerts #68/#69 (pin enhanced-resolve)#138

Merged
laileni-aws merged 1 commit into
aws:mainfrom
laileni-aws:fix/dependabot-security-alerts-part3
Jul 15, 2026
Merged

deps: fix webpack SSRF alerts #68/#69 (pin enhanced-resolve)#138
laileni-aws merged 1 commit into
aws:mainfrom
laileni-aws:fix/dependabot-security-alerts-part3

Conversation

@laileni-aws

Copy link
Copy Markdown
Contributor

Problem

The webpack SSRF alerts #68 / #69 were deferred from #136 / #137 because
bumping webpack to the patched ^5.104.1 pulls enhanced-resolve 5.24.x,
whose stricter exports-field resolution breaks the web bundle
(aws-core-vscode/shared no longer resolves from packages/core/dist), failing
the package VSIX and test Web jobs.

Solution

Bump webpack to ^5.104.1 (resolves #68/#69) and override
enhanced-resolve to 5.17.1 — the exact resolver main uses, which is
known to resolve core's exports correctly. npm overrides bypass webpack's
declared enhanced-resolve range, so the SSRF fix lands while the working,
lenient resolver is retained.

Stacked on #137 (and #136). Contains the category-1 and category-2 fixes
plus this resolver fix; net-new here is the deps: fix webpack SSRF alerts
commit. Brings the total to 154 / 160 resolved.

Change

Package Change Alerts
webpack ^5.95.0^5.104.1 #68, #69 (build-time SSRF)
enhanced-resolve (override) pinned 5.17.1 keeps web-build exports resolution working

Still open after this PR (6 — none safely fixable here)

Alert(s) Reason
aws-sdk #42, #44 No patched version (v2, maintenance)
elliptic #43 No patched version (<=6.6.1, none released)
@opentelemetry/core #225 OTel 2.0.0 family pinned by @aws/language-server-runtimes; needs that upgraded
@smithy/config-resolver #41 Old smithy pinned by old @aws-sdk v3 clients; needs those upgraded
@protobufjs/utf8 #200 Stale/false positive — already fixed via protobufjs@7.6.3

Verification

Draft: opened as a draft to validate the resolver pin against the web build.


  • Treat all work as PUBLIC. Private feature/x branches will not be squash-merged at release time.
  • Your code changes must meet the guidelines in CONTRIBUTING.md.
  • License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

Problem:
Bumping webpack to the patched ^5.104.1 (fixes GHSA build-time SSRF
alerts aws#68/aws#69) pulls enhanced-resolve 5.24.x, whose stricter exports
resolution breaks the web bundle (aws-core-vscode/shared no longer
resolves from packages/core/dist), failing package VSIX and test Web.

Solution:
Bump webpack to ^5.104.1 and override enhanced-resolve to 5.17.1 (the
version main uses and known to resolve core's exports). npm overrides
bypass webpack's declared enhanced-resolve range, so the SSRF fix lands
while the lenient, working resolver is retained. Brings total resolved
to 154 of 160 Dependabot alerts.

Remaining 6 alerts are unpatched upstream (aws-sdk v2, elliptic),
require major upstream package upgrades (@opentelemetry/core via
@aws/language-server-runtimes, @smithy/config-resolver via old @aws-sdk
clients), or are stale/false-positive (@protobufjs/utf8).
@laileni-aws
laileni-aws force-pushed the fix/dependabot-security-alerts-part3 branch from 95c1dd6 to f3dab58 Compare July 14, 2026 20:36
@laileni-aws
laileni-aws marked this pull request as ready for review July 14, 2026 20:40
@laileni-aws
laileni-aws requested a review from a team as a code owner July 14, 2026 20:40
@laileni-aws
laileni-aws enabled auto-merge (squash) July 15, 2026 00:02
@laileni-aws
laileni-aws merged commit 73bdc86 into aws:main Jul 15, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants