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

duplex option is required when sending a body. #1963

Closed
ronag opened this issue Feb 22, 2023 · 4 comments · Fixed by #1965
Closed

duplex option is required when sending a body. #1963

ronag opened this issue Feb 22, 2023 · 4 comments · Fixed by #1965
Labels
bug Something isn't working

Comments

@ronag
Copy link
Member

ronag commented Feb 22, 2023

https://github.com/nodejs/undici/actions/runs/4241282379/jobs/7371309085

Error: or [Error]: RequestInit: duplex option is required when sending a body.
    at fetch (/home/runner/work/undici/undici/index.js:109:13)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

@ronag ronag added the bug Something isn't working label Feb 22, 2023
@ronag
Copy link
Member Author

ronag commented Feb 22, 2023

@KhafraDev

@KhafraDev
Copy link
Member

unfortunately the same issue from web-platform-tests/wpt#36268

@ronag
Copy link
Member Author

ronag commented Feb 22, 2023

Can we disable/fix it in our copy?

@KhafraDev
Copy link
Member

There's no way to disable a single test (at least to my knowledge, they're not supposed to crash the runner if they fail), so the alternative is to disable the file altogether (disables many tests) or to patch it. I patched it previously, but the change was undone in #1954. I'll look for a way to fix it in the runner, or patch it again if I can't figure anything out

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants