Skip to content

When graceful shutdown, ensure the streams are processed before closing connection#2736

Merged
violetagg merged 2 commits into1.0.xfrom
issue-2735
Mar 22, 2023
Merged

When graceful shutdown, ensure the streams are processed before closing connection#2736
violetagg merged 2 commits into1.0.xfrom
issue-2735

Conversation

@violetagg
Copy link
Copy Markdown
Member

Fixes #2735

@violetagg violetagg added the type/bug A general bug label Mar 22, 2023
@violetagg violetagg added this to the 1.0.31 milestone Mar 22, 2023
@violetagg violetagg requested a review from a team March 22, 2023 09:15
@violetagg violetagg changed the title When graceful shutdown, ensure the streams are process before closing connection When graceful shutdown, ensure the streams are processed before closing connection Mar 22, 2023
Copy link
Copy Markdown
Contributor

@pderop pderop left a comment

Choose a reason for hiding this comment

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

ok, nice patch, looks good to me.

@violetagg
Copy link
Copy Markdown
Member Author

HttpClientTest > testHttpClientCancelled() is failing on Windows OS. I checked the test and it does not enable graceful shutdown. So my opinion is that this failure is not because of this change.

https://github.com/reactor/reactor-netty/actions/runs/4488211188/jobs/7893282178#step:5:157

@violetagg violetagg merged commit 5dd2034 into 1.0.x Mar 22, 2023
@violetagg violetagg deleted the issue-2735 branch March 22, 2023 13:31
violetagg added a commit that referenced this pull request Mar 22, 2023
violetagg added a commit that referenced this pull request Mar 22, 2023
violetagg added a commit that referenced this pull request Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/bug A general bug

Projects

None yet

3 participants