Issue #12272 - Potential deadlock with Vaadin.

Updated test case to use Input/OutputStream as well as Reader/Writer. Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
jetty · Nov 8, 2024 · 9f8cc61 · 9f8cc61
1 parent 473445b
commit 9f8cc61
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/...s/src/test/java/org/eclipse/jetty/ee10/test/client/transport/Http2AsyncIOServletTest.java b/...s/src/test/java/org/eclipse/jetty/ee10/test/client/transport/Http2AsyncIOServletTest.java
@@ -49,7 +49,6 @@
 import org.eclipse.jetty.util.FuturePromise;
 import org.eclipse.jetty.util.component.LifeCycle;
 import org.junit.jupiter.api.AfterEach;
-import org.junit.jupiter.api.Test;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.ValueSource;
 
@@ -223,8 +222,9 @@ public void onStartAsync(AsyncEvent event)
         assertTrue(errorLatch.await(5, TimeUnit.SECONDS));
     }
 
-    @Test
-    public void testSessionShutdownWithPendingRequest() throws Exception
+    @ParameterizedTest
+    @ValueSource(booleans = {false, true})
+    public void testSessionShutdownWithPendingRequest(boolean useReaderWriter) throws Exception
     {
         CountDownLatch serverFailureLatch = new CountDownLatch(1);
         start(new HttpServlet()
@@ -234,7 +234,10 @@ protected void service(HttpServletRequest request, HttpServletResponse response)
             {
                 try
                 {
-                    request.getInputStream().transferTo(response.getOutputStream());
+                    if (useReaderWriter)
+                        request.getReader().transferTo(response.getWriter());
+                    else
+                        request.getInputStream().transferTo(response.getOutputStream());
                 }
                 catch (Throwable x)
                 {