[ 
https://issues.apache.org/jira/browse/AMQ-6521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15735487#comment-15735487
 ] 

Christopher L. Shannon commented on AMQ-6521:
---------------------------------------------

Oops, I thought I had tested this properly but apparently not. Looks like when 
I reverted the previous commit it had restored the check for Jetty 8.x and 
Jetty 9.2 but not Jetty 9.3.  I retested this again and there is actually one 
more change that is needed for this to work with Jetty 9.3.  I will push a new 
commit with the changes.

Yes it is difficult to test right now because our broker pulls in only Jetty 
9.2.x.  So our tests just run against 9.2.x as that is the main version 
supported in 5.14.x.  In 5.15.x I am planning to upgrade to 9.3 so that will 
get all the tests run against it then.  

To run tests against both 9.2 and 9.3 we'd probably need to create another 
subproject and override the Jetty dependency to be 9.3 and re-run the Http 
tests. That is something that could be looked at in 5.15.0 if we decide to 
provide support for more than one version of Jetty besides just 9.3.0

> compatibility issue with jetty 9.3.11 in 
> org.apache.activemq.transport.http.HttpTransportServer
> -----------------------------------------------------------------------------------------------
>
>                 Key: AMQ-6521
>                 URL: https://issues.apache.org/jira/browse/AMQ-6521
>             Project: ActiveMQ
>          Issue Type: Bug
>    Affects Versions: 5.14.1
>            Reporter: Carsten Hammer
>            Assignee: Christopher L. Shannon
>            Priority: Minor
>             Fix For: 5.15.0, 5.14.2
>
>
> There is a instantiation of a class that does not exists, see 
> org.apache.activemq.transport.http.HttpTransportServer.java:
>  private void addGzipHandler(ServletContextHandler contextHandler) throws 
> Exception {
>         Handler handler = new GzipHandler();
>         contextHandler.setHandler(handler);
>     }
> org.eclipse.jetty.servlets.gzip.GzipHandler does not exist. Instead there is 
> a class org.eclipse.jetty.server.handler.gzip.GzipHandler.java
> in 
> https://mvnrepository.com/artifact/org.eclipse.jetty/jetty-server/9.3.13.v20161014
> Because of this activemq is not compatible with jetty versions since 9.0



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to