Skip to content

[8.12] Fix race in HTTP response shutdown handling (#105306)#105327

Merged
elasticsearchmachine merged 1 commit intoelastic:8.12from
DaveCTurner:backport/8.12/pr-105306
Feb 9, 2024
Merged

[8.12] Fix race in HTTP response shutdown handling (#105306)#105327
elasticsearchmachine merged 1 commit intoelastic:8.12from
DaveCTurner:backport/8.12/pr-105306

Conversation

@DaveCTurner
Copy link
Copy Markdown
Member

Backports the following commits to 8.12:

Similar to elastic#97301, the fix in elastic#105293 was still not quite correct: we
could in principle shut down the transport after checking `isOpen()` but
before sending the message. Applying the same fix as for the transport
layer here.
@DaveCTurner DaveCTurner added :Distributed/Network Http and internode communication implementations >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Distributed Meta label for distributed team. labels Feb 9, 2024
@elasticsearchmachine elasticsearchmachine merged commit 3bab909 into elastic:8.12 Feb 9, 2024
@DaveCTurner DaveCTurner deleted the backport/8.12/pr-105306 branch February 9, 2024 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug :Distributed/Network Http and internode communication implementations Team:Distributed Meta label for distributed team. v8.12.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants