Skip to content

Conversation

@wbamberg
Copy link
Collaborator

@wbamberg wbamberg commented Aug 20, 2025

Follow-up to #40782 and #40789.

Main point being that you can't use the signal option to close an already-established connection, you can only use it to cancel a connection attempt before it has been established.

cc @tomayac .

This PR:

@wbamberg wbamberg requested a review from a team as a code owner August 20, 2025 17:45
@wbamberg wbamberg requested review from sideshowbarker and removed request for a team August 20, 2025 17:45
@github-actions github-actions bot added Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed labels Aug 20, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Aug 20, 2025

Copy link
Contributor

@tomayac tomayac left a comment

Choose a reason for hiding this comment

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

Non-owner LGTM, thank you!

@sideshowbarker sideshowbarker merged commit 7a418e5 into mdn:main Aug 24, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants