Skip to content

feat: abort request.signal when the response closes prematurely - #16793

Merged
Rich-Harris merged 2 commits into
version-3from
request-signal-response-close
Aug 13, 2026
Merged

feat: abort request.signal when the response closes prematurely#16793
Rich-Harris merged 2 commits into
version-3from
request-signal-response-close

Conversation

@Nic-Polumeyv

Copy link
Copy Markdown
Contributor

A request whose body has been fully read never aborts its signal on client disconnect, because readableAborted requires the stream to die before end. Give getRequest the response so disconnects are detected on the response side instead — writableEnded rather than writableFinished because HTTP/2 marks cancelled streams as finished. Split out of #16790.

@pkg-svelte-dev

Copy link
Copy Markdown

Install the latest version of @sveltejs/kit from c984d07:

pnpm add https://pkg.svelte.dev/@sveltejs/kit/c/c984d078ba52ece748ffdb3c2953efa2c34efc31

Open in pkg.svelte.dev: https://pkg.svelte.dev/repos/kit/pr/16793

@changeset-bot

changeset-bot Bot commented Aug 13, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: c984d07

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@sveltejs/kit Minor
@sveltejs/adapter-node Patch

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

@svelte-docs-bot

Copy link
Copy Markdown

@Rich-Harris
Rich-Harris merged commit 562ae23 into version-3 Aug 13, 2026
27 checks passed
@Rich-Harris
Rich-Harris deleted the request-signal-response-close branch August 13, 2026 22:40
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.

2 participants