-
Notifications
You must be signed in to change notification settings - Fork 89
[Bug]: getPageResolver
generating incorrect paths on Windows
#941
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
Comments
Hey. Thanks for the report. I think I am going to need a repro from you, because I've failed to reproduce it myself. See #943 and the deploy preview |
I'm thinking it might be due to it being in a monorepo. It is a Sanity project so we have the following structure:
Perhaps the Netlify function isn't able to bundle the packages from the monorepo properly? I should have some time free tomorrow to try and investigate and put together a reproduction. 🤞 |
Thank you. It shouldn't be a problem with it being a monorepo, as we have several monorepo test sites, but it may still be related. A repro would be useful. It would also be helpful if you could run |
The pages.js file from running the
|
We're using Azure DevOps for our CI process and decided to try running the same build/release process on a Unix agent and it appears to have resolved the issue for us. There must be something going awry on Windows agents specifically. |
Oh yes that does look like it! Thanks for the report. I will rename this issue and prioritise a fix. |
getPageResolver
generating incorrect paths on Windows
Can you confirm which version of Node is being used in the build? It would be useful if you could share the build logs too. We run integration tests on Windows which include snapshots of the pages.js file, so it does seem to generate the correct ones there at least. |
Sorry just checked back in on this, the node version would have been 14.16.0. To be honest it should have been updated to use the latest LTS but that was how the release was setup when we last ran it on the Windows agent. I can share the log directly if you're able to provide a contact email or similar. Would prefer not to post the entire log on here just in case. |
I have a probable solution. I'll see if I can get that in a release today. |
This should be fixed in rc.2 |
@ascorbic I've updated to rc.2 and ran the release process on the Windows agent and we're getting the entire site timing out with this error now.
|
@rshackleton oh dear. Can you share the |
We're ok using the hosted Ubuntu agents for now, and we're looking at setting up our own dedicated Ubuntu agent to run alongside our Windows agents so that should be fine for us in the long run. I'll try and get the pages.js file and logs to you but we're currently trying to fix the stack overflow issue that has come up with installing the latest Netlify CLI. 😫 |
Would that be this issue? |
Yep, we were using the CLI via a global install so we're updating the CI to use it via a dev dependency. That is fixed now and I think the timeout issue might actually be due to me enabling ISR earlier on - I wasn't able to properly check that was working properly because of the release issues. 🤦♂️ Just undoing those changes so we can verify it is working on the Ubuntu agent again, then I can try it on the Windows agent and see how it works out. It has been a bit of a day. 😅 |
Ok, so we're back up and running. Tested the build on rc.2 on both Ubuntu and Windows agent and both are working great now, no more dependency issue even on the Windows agent! |
Oh wow, that's great news! Thanks so much for your help in tracking this down. |
No worries, cheers for getting a fix out so promptly! |
Uh oh!
There was an error while loading. Please reload this page.
Steps to reproduce
When using the latest RC version of the plugin all SSR pages in Next.js are failing to find the
styled-components
dependency imported in_document.ts
.This is happening with and without the esbuild function bundle option. The repo is a private repo so I cannot share this currently, I can see about trying to put together a reproduction repo but that will take some time.
A link to a reproduction repository
No response
Plugin version
^4.0.0-rc.1
More information about your build
netlify.toml
)What OS are you using?
Windows
Your
netlify.toml
file`netlify.toml`
Relevant log output (or link to your logs)
Build logs
The text was updated successfully, but these errors were encountered: