Since undertowHttpServer conflicts with other HTTPServers. Temporarily bypass with transferToForUndertow. It's better than return and pass.

Comment From: pivotal-cla

@JiangTChen Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

Comment From: pivotal-cla

@JiangTChen Thank you for signing the Contributor License Agreement!

Comment From: sbrannen

Hi @JiangTChen,

Thanks for submitting your first PR to the Spring Framework!

Since undertowHttpServer conflicts with other HTTPServers. Temporarily bypass with transferToForUndertow.

Can you briefly expound on the first statement and explain how your workaround (the second statement) solves the problem?

Comment From: sbrannen

Please note that your PR fails the build:

MultipartIntegrationTests > transferToWithUndertow() FAILED
    java.lang.AssertionError at MessageFormatter.java:115

In light of that, I'm marking this as a draft.

Comment From: JiangTChen

@sbrannen Thanks for your review. I found that the pass rate was not 100%. You can use @RepeatedTest(100) instead of @Test to verify it. There are three kinds of exceptions. I think the root cause is about "Connection prematurely closed BEFORE response". It's very tricky, I'm not sure if it's a configuration of AbstractHttpHandlerIntegrationTests#startServer or undertow bug. I will keep digging. Should we add @Disable on transferToWithUndertow() first, to instead of return pass. It can alter others and help to fix.

Thanks

Comment From: sbrannen

I found that the pass rate was not 100%. You can use @RepeatedTest(100) instead of @test to verify it. There are three kinds of exceptions. I think the root cause is about "Connection prematurely closed BEFORE response". It's very tricky, I'm not sure if it's a configuration of AbstractHttpHandlerIntegrationTests#startServer or undertow bug. I will keep digging.

Thanks for the explanation.

Should we add @Disable on transferToWithUndertow() first, to instead of return pass. It can alter others and help to fix.

I originally thought you were proposing a fix for Undertow, but since that's not the case, yes adding @Disabled to the new test method is a must.

And please do continue investigating the root cause of the Undertow issue. We haven't had the time to do that ourselves, and we'd be grateful for a fix from the community.

Comment From: JiangTChen

Hi @sbrannen, Please help to review again. Thanks.

Comment From: sbrannen

This has been merged into main.

Feel free to continue to investigate the cause of the issue and report your findings in #25310.

Thanks