Disable Http2RSTFloodProtectionTest on Windows

quarkusio · Oct 27, 2023 · b82f9ee · b82f9ee
1 parent d110270
commit b82f9ee
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/...ttp/deployment/src/test/java/io/quarkus/vertx/http/http2/Http2RSTFloodProtectionTest.java b/...ttp/deployment/src/test/java/io/quarkus/vertx/http/http2/Http2RSTFloodProtectionTest.java
@@ -14,6 +14,8 @@
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.Assumptions;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.condition.DisabledOnOs;
+import org.junit.jupiter.api.condition.OS;
 import org.junit.jupiter.api.extension.RegisterExtension;
 
 import io.quarkus.test.QuarkusUnitTest;
@@ -29,6 +31,7 @@
 /**
  * Reproduce CVE-2023-44487.
  */
+@DisabledOnOs(OS.WINDOWS)
 public class Http2RSTFloodProtectionTest {
 
     @TestHTTPResource(value = "/ping", ssl = true)
@@ -81,7 +84,7 @@ void run(HttpClient client, int port, boolean plain) throws InterruptedException
                     .compose(HttpClientRequest::send);
         }
 
-        for (int i = 0; i < 250; i++) { // must be higher thant the NEtty limit (200 / 30s)
+        for (int i = 0; i < 250; i++) { // must be higher than the Netty limit (200 / 30s)
             client.request(GET, port, "localhost", "/ping")
                     .onSuccess(req -> req.end().onComplete(v -> req.reset()));
         }