-
Notifications
You must be signed in to change notification settings - Fork 525
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
Undici throws a terminated error #1490
Comments
Thanks for reporting. |
@Eddy84 I'm experiencing the same issue, did you manage to solve this? |
Downgrading node version from 18 to 16 (stable) did the trick for me. |
Same issue, but already on v16.14.0, v16.16.0 made no difference and it was working last week. |
@LiamKarlMitchell did you found any solution for that? |
@hpiaia Check my comment in this other issue dotansimha/graphql-code-generator#7934 (comment) for a workaround. |
Options request shouldn't have a Content-Length set.
Below is a middleware, which can be added to App/Http/Middlewares and put
into lighthouse middlewares config array.
It does add it to every POST request however for the graphql endpoint.
https://gist.github.com/LiamKarlMitchell/18f1e43ae6b772864d1b6a6ec1f71ba9
Works for me for a temp work-around until this can be resolved upstream.
…On Thu, 28 Jul 2022, 11:05 am Thomas Aull, ***@***.***> wrote:
@hpiaia <https://github.com/hpiaia> Check my comment in this other issue dotansimha/graphql-code-generator#7934
(comment)
<dotansimha/graphql-code-generator#7934 (comment)>
for a workaround.
—
Reply to this email directly, view it on GitHub
<#1490 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAPAP3INOG45VBBWVQLJTMTVWG6ERANCNFSM5YRKXXXA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@thomasaull @LiamKarlMitchell,
Reference: |
I'm closing this as |
@mcollina I'd be happy to provde a reproduction. However since this is happening in a mixedx project with php laravel and javascript, I'm not sure what would be the best way to do this? |
Try to reproduce with just undici and Node.js. Things that work well:
|
@mcollina Had no luck reproducing it with node :-/ |
happens periodically for us as well using node 18 with undici 5.22.1. No clear reproducer but we see it daily. |
Happens to us, using Node 20 in a Google Cloud Functions, however works fine locally |
Happens also periodically for me using node 20 and nest.js - No clear reproducer but we see it daily. |
+1, Affected by this, inconsistently, no defined cause. |
Happens when IPv6 is disabled. Reproduce on Ubuntu:
Fetch usage with node results in exit code |
Bug Description
I have a bug with graphl-codegen which is very likely caused by undici closing the connection and sending a terminated signal?
I have double checked my backend with various graphql clients and plain http requests doing the introspection query, the bug is definitely in undici.
The server receives the request, but it is terminated.
A stacktrace is included below:
Here is the related issue
dotansimha/graphql-code-generator#7934
Reproducible By
See the linked issue above
Expected Behavior
Return a response
Logs & Screenshots
See below
Environment
Additional context
Stacktrace:
The text was updated successfully, but these errors were encountered: