Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UNDERTOW-2275 Suspend reads before terminating a request to cleanup read-timeout #1486

Merged
merged 1 commit into from
Aug 17, 2023

Conversation

TomasHofman
Copy link
Contributor

@TomasHofman TomasHofman force-pushed the UNDERTOW-2275-2.2.x branch 2 times, most recently from 5ebade8 to ad4fca7 Compare June 9, 2023 18:32
@TomasHofman
Copy link
Contributor Author

There are two test failures in the CI jobs: HttpContinueSslServletTestCase, SameSiteCookieHandlerTestCase. Both are buffer leak issues, I think they are intermittent as I couldn't reproduce them locally.

@fl4via
Copy link
Member

fl4via commented Jun 21, 2023

@TomasHofman there is a failure in the CI that needs to be checked.

@fl4via fl4via added waiting PR update Awaiting PR update(s) from contributor before merging failed CI Introduced new regession(s) during CI check backport The PR is the result of backporting another PR to a maintainance branch labels Jun 21, 2023
@TomasHofman
Copy link
Contributor Author

@fl4via sorry, that was trailing spaces in source file...

Now there are two test failures which look unrelated to me.

@fl4via fl4via removed waiting PR update Awaiting PR update(s) from contributor before merging failed CI Introduced new regession(s) during CI check labels Aug 17, 2023
@fl4via fl4via merged commit 17647fd into undertow-io:2.2.x Aug 17, 2023
32 of 34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport The PR is the result of backporting another PR to a maintainance branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants