Skip to content

goog-vulnz flags CVE-2024-24790 in esbuild 0.19.7 #4045

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
paulmarsicloud opened this issue Jan 30, 2025 · 3 comments · Fixed by #4046
Closed

goog-vulnz flags CVE-2024-24790 in esbuild 0.19.7 #4045

paulmarsicloud opened this issue Jan 30, 2025 · 3 comments · Fixed by #4046

Comments

@paulmarsicloud
Copy link
Contributor

paulmarsicloud commented Jan 30, 2025

Hello!

Artifact Registry flags CVE-2024-24790 in drizzle-kit usage of esbuild (e.g. /app/node_modules/drizzle-kit/node_modules/@esbuild/linux-x64/bin/esbuild)

Upon closer inspection it seems that per evanw/esbuild#4022 (comment) esbuild 0.24.2 should resolve this one. I'll make an issue and attempt a PR to bump this for ya'll 👍

@AndriiSherman
Copy link
Member

Fixed in [email protected]

@anthonyhagi
Copy link

It looks like it's not completely resolved. Since there's a reliance on @esbuild-kit/[email protected], npm still installs esbuild@~0.18.20. Is it possible to move away from @esbuild-kit/core-utils or to update it to a newer version? I believe it would solve the issue.

@sterkmi
Copy link

sterkmi commented Apr 17, 2025

It seems the issue is this dependency in drizzle-kit: "@esbuild-kit/esm-loader": "^2.5.5", , This repo is archived and there is the hint that the functionality was merged in tsx:
https://github.com/esbuild-kit/esm-loader?tab=readme-ov-file#%EF%B8%8F-project-moved-repository-unmaintained

This project has been merged with tsx and this repository is no longer maintained.

Use tsx instead: node --loader tsx/esm ./file.ts

esm-loader is used here:

js: `#!/usr/bin/env -S node --loader @esbuild-kit/esm-loader --no-warnings`,

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 a pull request may close this issue.

4 participants