Skip to content
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

RangeError: Maximum call stack size exceeded - For exports parsed into a lot of nodes #402

Open
BenjaminBruenau opened this issue Mar 29, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@BenjaminBruenau
Copy link

BenjaminBruenau commented Mar 29, 2024

TL;DR

Recursive call of walk in static-eval.ts for an export that is parsed into a lot of nodes leads to:

RangeError: Maximum call stack size exceeded

I am not sure whether this needs to be addressed as it is quite the edge case and probably very few cases will arise where something is exported that is leading to so many recursive calls and thus this error.

Background

I am deploying a Nuxt 3 Application to an arm64 server via a docker image, which worked fine until a couple of days ago when I was suddenly confronted with this error when making the production build
(more specifically when the build of Nuxt's Node.js server [nitro] was created).

RangeError: Maximum call stack size exceeded

Full Error Output
RangeError: Maximum call stack size exceeded

Exception in PromiseRejectCallback:
/app/node_modules/@vercel/nft/out/utils/static-eval.js:69
         let l = await walk(node.left);
                       ^

RangeError: Maximum call stack size exceeded


[nitro]  ERROR  RangeError: Maximum call stack size exceeded


undefined

at walk (node_modules/@vercel/nft/out/utils/static-eval.js:18:28)
at Object.BinaryExpression (node_modules/@vercel/nft/out/utils/static-eval.js:69:23)
at walk (node_modules/@vercel/nft/out/utils/static-eval.js:18:28)
at Object.BinaryExpression (node_modules/@vercel/nft/out/utils/static-eval.js:69:23)
at walk (node_modules/@vercel/nft/out/utils/static-eval.js:18:28)
at Object.BinaryExpression (node_modules/@vercel/nft/out/utils/static-eval.js:69:23)
at walk (node_modules/@vercel/nft/out/utils/static-eval.js:18:28)
at Object.BinaryExpression (node_modules/@vercel/nft/out/utils/static-eval.js:69:23)
at walk (node_modules/@vercel/nft/out/utils/static-eval.js:18:28)

Reason

After doing a little digging and debugging I found out the error occurred while evaluating the following export of mysql2

https://github.com/sidorares/node-mysql2/blob/a9c6c3eab0691ee03482514080d70d90344d0326/lib/constants/ssl_profiles.js

which was changed in sidorares/node-mysql2#2131 and introduced a larger cert file wich is later parsed/traversed in https://github.com/vercel/nft/blob/main/src/utils/static-eval.ts .

They seem to have done a little oopsie when updating the certificates as it is a single big string (composed of a lot of concats)
inside an array instead of separating them one by one as array elements.

I guess it resolves all the concat operations for each node parsed from the single, large array element (so I think for each node the whole mash-up of certificate-string-concats is traversed with walk recursively or something similar like that) leading to the stack-size of 864 kBytes on arm64 machines being violated (as opposed to the stack size of 984 kBytes on amd64 machines where this error is not thrown)

Workaround

I downgraded to [email protected] for the time being.

It is also possible to change the stack-size via an --v8-optionsnode flag called --stack-size
and call e.g. yarn build like so:

node --stack-size=984 `$(which yarn) build`

as it is not possible to set it via the NODE_OPTIONS ENV Variable (not allowed by node)

Not sure what effects this could have in the long term as there seems to be good reason for the stack-size being smaller for arm64 machines.

Interesting read on this topic: nodejs/node#41163

The recommended solution seems to be to rewrite recursive functions.

Steps to reproduce:

  1. npx nuxi@latest init [project-name]
  2. yarn add [email protected]
  3. build application for production:
    run yarn install && yarn build on an arm64 machine
    or run docker build --platform linux/arm64 . with a Dockerfile like the following:
Dockerfile
FROM node:20.11-alpine as builder

WORKDIR /app
COPY package.json ./
COPY nuxt.config.ts ./
RUN yarn set version 1.22.19
RUN yarn config set network-timeout 600000 -g & yarn install
COPY . .
RUN yarn build

FROM node:20.11-alpine

COPY --from=builder /app/.output /test/.output

WORKDIR /test/.output

ENV NUXT_HOST=0.0.0.0
ENV NUXT_PORT=3000
EXPOSE 3000

CMD ["node", "/test/.output/server/index.mjs"]
@styfle
Copy link
Member

styfle commented Apr 2, 2024

Thanks for reporting this issue!

I was able to reproduce with a single file

curl -JLO "https://raw.githubusercontent.com/sidorares/node-mysql2/675dd0411017053375a18248c38a2b8b55de24ed/lib/constants/ssl_profiles.js"
npm install -g @vercel/nft
nft print ssl_profiles.js

It looks like you have already dug deep on this, so feel free to submit a PR if you have a fix, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants