-
Notifications
You must be signed in to change notification settings - Fork 28.8k
Match subrequest handling for edge and node #77477
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
Conversation
@ijjk was there a breaking change from 13.5.8/13.5.9 to 13.5.10? cc @ztanner @huozhi @feedthejim My project depends on
Upon investigation I saw that it only happens in |
Hi, could you share the platform you're building on and your |
@ijjk using Docker. Unfortunately the next info command fails due to SIGBUS. See output below. I also included an open source repository that reproduces the problem from a generated boilerplate app. Output
Reproduction in a RepositoryI made a simple repository where I installed NextJS using https://github.com/rmartins0/next-13-5-10-sigbus You can clone the repository and run
Then you will get the SIGBUS issue:
Running just npx next build:
Other Docker imagesI also tried and reproduced on images It was not reproduced on image |
Same issue here. |
Same issue using Nx 16 and Node 18/20.
Using Next |
Hi, we just published |
Thank you @ijjk ! Tested on my nx env and @rmartins0 's repro repo and confirmed working. ❯ git remote -v ─╯
origin https://github.com/rmartins0/next-13-5-10-sigbus.git (fetch)
origin https://github.com/rmartins0/next-13-5-10-sigbus.git (push)
❯ DOCKER_BUILDKIT=0 docker build . ─╯
DEPRECATED: The legacy builder is deprecated and will be removed in a future release.
BuildKit is currently disabled; enable it by removing the DOCKER_BUILDKIT=0
environment-variable.
Sending build context to Docker daemon 261.3MB
Step 1/7 : FROM node:21-alpine
21-alpine: Pulling from library/node
94747bd81234: Already exists
c6afd58f95b7: Already exists
c816472f4f4e: Already exists
0fbe816ec3aa: Already exists
Digest: sha256:78c45726ea205bbe2f23889470f03b46ac988d14b6d813d095e2e9909f586f93
Status: Downloaded newer image for node:21-alpine
---> c3d7eb7b3b2b
Step 2/7 : WORKDIR /abc
---> Running in 45dfa08c18f8
---> Removed intermediate container 45dfa08c18f8
---> f484ce4e8be5
Step 3/7 : COPY . /abc
---> b0a1a1df0299
Step 4/7 : RUN npm install
---> Running in b9bfca9377bb
added 4 packages, and audited 308 packages in 2s
122 packages are looking for funding
run `npm fund` for details
1 high severity vulnerability
To address all issues (including breaking changes), run:
npm audit fix --force
Run `npm audit` for details.
npm notice
npm notice New major version of npm available! 10.5.0 -> 11.2.0
npm notice Changelog: <https://github.com/npm/cli/releases/tag/v11.2.0>
npm notice Run `npm install -g [email protected]` to update!
npm notice
---> Removed intermediate container b9bfca9377bb
---> 9cab799c0f6c
Step 5/7 : RUN npx next info --verbose
---> Running in 6a4cd3e85f8e
- Host system information: pass
- Next.js installation: pass
- Node.js diagnostic report: pass
- next-swc installation: pass
- next-swc shared object dependencies: skipped
This diagnostics uses system-installed tools (ldd) to check next-swc dependencies, but it is not found. Skipping dependencies check.
Generated diagnostics report
Please copy below report and paste it into your issue.
### Host system information
WSL: false
Docker: true
CI: false
### Next.js installation
Binaries:
Node: 21.7.3
npm: 10.5.0
Yarn: 1.22.19
pnpm: N/A
Relevant Packages:
next: 13.5.11
eslint-config-next: 13.5.8
react: 18.3.1
react-dom: 18.3.1
typescript: 5.8.2
Next.js Config:
output: N/A
...
... |
Backports #77474 to v13