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

Node server crashing when upgrade to 1.8 (@remix-run/web-fetch@4.3.2) #4737

Closed
nicksrandall opened this issue Dec 1, 2022 · 5 comments
Closed
Assignees

Comments

@nicksrandall
Copy link
Contributor

What version of Remix are you using?

1.8

Steps to Reproduce

If I have time, I'll try to create a minimal reproduction.

When I upgraded my app (deployed to fly.io as a Node service) to Remix 1.8 (from 1.7.6), I got alerts that my server was repeatedly crashing. The errror returned comes from @remix-run/web-fetch@4.3.2 https://github.com/remix-run/web-std-io/blob/main/packages/fetch/src/fetch.js#L342 where the error message says "Premature Close".

Reverting back to @remix-run/node@1.7.6 and @remix-run@web-fetch@4.31 solved the issue for me.

Expected Behavior

No fetch errrors.

Actual Behavior

Fetch errors.

@nicksrandall
Copy link
Contributor Author

CC @jacob-ebey

@jacob-ebey
Copy link
Member

Do you have a minimal example of the application code that is causing this codepath?

@leimantas
Copy link

@jacob-ebey
Can you check this?
#4741 (comment)

@nicksrandall
Copy link
Contributor Author

Same issue as #4993

@nicksrandall
Copy link
Contributor Author

I'll close this issue for now as the other ticket has a failing test case that can be used to debug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants