This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tomcat-tck.git

commit 9e3481800164f8eb1806cea7f36556b3fd98e3d0
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Wed Mar 20 10:03:26 2024 +0000

    We don't need HTTP/2 support for TCK
    
    I added HTTP/2 support for the TCK tests so I could run the TCK's HTTP/2
    tests. Since the TCK's HTTP/2 tests all depend on HTTP/2 push - which
    Tomcat no longer implements - there is no need to retain HTTP/2 support
    here.
---
 .../tomcat/tck/servlet/TomcatServletTckConfiguration.java    | 12 ------------
 1 file changed, 12 deletions(-)

diff --git 
a/servlet-tck/src/test/java/org/apache/tomcat/tck/servlet/TomcatServletTckConfiguration.java
 
b/servlet-tck/src/test/java/org/apache/tomcat/tck/servlet/TomcatServletTckConfiguration.java
index 2b46a62..664aaec 100644
--- 
a/servlet-tck/src/test/java/org/apache/tomcat/tck/servlet/TomcatServletTckConfiguration.java
+++ 
b/servlet-tck/src/test/java/org/apache/tomcat/tck/servlet/TomcatServletTckConfiguration.java
@@ -72,18 +72,6 @@ public class TomcatServletTckConfiguration implements 
LoadableExtension {
                 portField.setAccessible(true);
                 portField.set(configuration, Integer.valueOf(localPort));
 
-                   // Add HTTP/2 support
-                   Http2Protocol http2Protocol = new Http2Protocol();
-                   AbstractHttp11Protocol<?> httpProtocol = 
(AbstractHttp11Protocol<?>) connector.getProtocolHandler();
-                   http2Protocol.setHttp11Protocol(httpProtocol);
-                   connector.addUpgradeProtocol(http2Protocol);
-                   // Upgrade protocols need to be added before 
Connector.init().
-                   // Can't do that so use reflection to achieve the same aim.
-                   Method m = AbstractHttp11Protocol.class.getDeclaredMethod(
-                               "configureUpgradeProtocol", 
UpgradeProtocol.class);
-                   m.setAccessible(true);
-                   m.invoke(httpProtocol, http2Protocol);
-
                    if 
("https".equals(System.getProperty("arquillian.launch"))) {
                        // Need to enabled HTTPS
                    }


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to