-
Notifications
You must be signed in to change notification settings - Fork 972
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
Cannot deploy Next.js app with Server Side Rendering #7146
Comments
This issue does not have all the information required by the template. Looks like you forgot to fill out some sections. Please update the issue with more information. |
I have seen this issue reported before: #6253 |
Also, seen here #5158 |
Seen similar issue here: #6435 |
Did some debugging. This seems happening because glob function is returning absolute paths instead of relative paths that are needed later in a copying process: firebase-tools/src/frameworks/next/utils.ts Lines 475 to 487 in e06ae7b
|
I can attempt a PR. |
Hey @dominoFire, thanks for reporting this. I was able to reproduce the error you encountered. I’ll raise this to our engineering team so they can take a look. To workaround the issue, could you try deploying using firebase-tools v13.8.0? It also looks like you may have identified the cause of the issue here, feel free to create a PR for this if you’re up for it! Contributions are always appreciated! |
@leoortizz PTAL too since you've been fighting some path fires |
Just noticed the glob package version used here is 7., new new glob version is 10., and it behaves differently with realpath parameter. On 7.* (in a test app) it return the full path on Windows if Line 124 in 143ffd0
|
thanks @dominoFire for reporting the issue and apologies for the inconvenience. I have a fix in #7148 and explained the reason in the PR description. As soon as that PR is merged and released this issue will be fixed. As a workaround, please use version |
[REQUIRED] Environment info
firebase-tools:
Platform: Windows 11, node --version: v20.10.0
[REQUIRED] Test case
This happened when I was trying to deploy a next.js app, noted in https://github.com/copilotkit/copilotkit
[REQUIRED] Steps to reproduce
Create a next app with
create-next-app
Add a next.js backend route. Example: src/app/route.ts, like so:
Init firebase Hosting in the created app
Deploy the project
[REQUIRED] Expected behavior
My app is deployed to Firebase
[REQUIRED] Actual behavior
The deploy fails with the following message:
The text was updated successfully, but these errors were encountered: