Skip to content

fix: apply timeout while parsing response body - #626

Draft
mmaxjr wants to merge 1 commit into
unjs:mainfrom
mmaxjr:fix/timeout-body-parsing
Draft

fix: apply timeout while parsing response body#626
mmaxjr wants to merge 1 commit into
unjs:mainfrom
mmaxjr:fix/timeout-body-parsing

Conversation

@mmaxjr

@mmaxjr mmaxjr commented Aug 20, 2026

Copy link
Copy Markdown

Resolves #620

This keeps the existing timeout signal active while the response body is being read.

Before this change, a response could be received successfully but then hang during body parsing if the body never completed. The new test covers that case with a response stream that starts but does not close.

@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

鈿欙笍 Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 152c6e11-c5ff-4f00-82e0-3e47fda55cfc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 馃攳 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

鉂わ笍 Share

Comment @coderabbitai help to get the list of available commands.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

timeout option doesn't timeout body parsing

1 participant