Skip to content

Conversation

@usualoma
Copy link
Member

fix: #279

There may be compatibility issues with other frameworks or existing implementations on different servers; however, calling cancel() appears to be the correct approach according to web standards.

Copy link
Member

@yusukebe yusukebe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@yusukebe
Copy link
Member

@usualoma

I see this makes sense. Thank you for handling it.

@yusukebe yusukebe merged commit 4b5a8d1 into honojs:main Sep 29, 2025
5 checks passed
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.

ReadableStream.cancel never trigger

2 participants