[Aikido] Fix security issue in esbuild via minor version upgrade from 0.28.0 to 0.28.1#1711
Open
aikido-autofix[bot] wants to merge 2 commits into
Open
[Aikido] Fix security issue in esbuild via minor version upgrade from 0.28.0 to 0.28.1#1711aikido-autofix[bot] wants to merge 2 commits into
aikido-autofix[bot] wants to merge 2 commits into
Conversation
AdamJHall
previously approved these changes
Jun 18, 2026
crispy101
previously approved these changes
Jun 18, 2026
f5cb085
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Upgrade esbuild to fix critical RCE vulnerability in Deno module (missing binary integrity verification), high-severity path traversal on Windows, and low-severity DoS issues.
✅ Code not affected by breaking changes.
✅ No breaking changes from esbuild 0.28.0 => 0.28.1 affect this codebase.
The codebase uses esbuild exclusively through the
buildSyncAPI for bundling Lambda functions during CDK synthesis. It does not use:esbuild's local development server - The code only uses
buildSync()for build-time bundling (found inpackages/constructs/esbuild/esbuild.ts), not theserve()API that would be affected by the backslash path restriction.Deno API - No Deno usage was found in the codebase, so the integrity check changes for custom package registries do not apply.
The HTTP path handling found in Lambda@Edge handlers (e.g., in
static-hosting,prerender-proxy) operates on CloudFront request URIs at runtime, which is unrelated to esbuild's development server.All breaking changes by upgrading esbuild from version 0.28.0 to 0.28.1 (CHANGELOG)
\backslash characters are no longer allowed in the local development serverNPM_CONFIG_REGISTRY) must now match the expected content due to integrity checks in the Deno API✅ 3 CVEs resolved by this upgrade
This PR will resolve the following CVEs:
servedirroot directory.🤖 Remediation details
Fix esbuild vulnerability: bump resolved version from 0.28.0 to 0.28.1
Short summary
This PR remediates security vulnerabilities in esbuild by refreshing the
yarn.locklockfile to resolveesbuildat0.28.1instead of0.28.0. The vulnerable package appears as a direct dependency in two workspace member manifests (packages/constructs/esbuild/package.jsonandpackages/cdk-aspects/package.json); onlyyarn.lockat the workspace root required a change.esbuild
esbuildis declared as a direct dependency ("esbuild": "^0.28.0") in bothpackages/constructs/esbuildandpackages/cdk-aspects. The^0.28.0semver range already admits0.28.1, so no manifest edits were required — the fix was a lockfile-only refresh viayarn up -R esbuild --mode=update-lockfile, which re-resolvedesbuildand all its@esbuild/*platform-specific optional packages from0.28.0to0.28.1. The bump was necessary because0.28.0is within the vulnerable range and0.28.1is the minimum patched version that addresses the reported advisories.Version changes