-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Do not send body with HEAD and GET on node integration #4105
Do not send body with HEAD and GET on node integration #4105
Conversation
🦋 Changeset detectedLatest commit: 8ee7555 The changes in this PR will be included in the next version bump. This PR includes changesets to release 12 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Related issue: #4104 |
I can relate the exact same issue here. I applyied your patch and seems to work now, thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense! Appreciate the fix.
This is a critical one so I'm going to merge even with the unrelated failure. |
Changes
Testing
With the default SSR it does not works and get error, after it works.
Error reported:
Docs