gemmellr commented on code in PR #5394:
URL: https://github.com/apache/activemq-artemis/pull/5394#discussion_r1880563231


##########
pom.xml:
##########
@@ -107,7 +107,7 @@
       <guava.version>33.3.1-jre</guava.version>
       <hawtio.version>2.17.7</hawtio.version>
       <jsr305.version>3.0.2</jsr305.version>
-      <jetty.version>10.0.24</jetty.version>
+      <jetty.version>12.0.15</jetty.version>

Review Comment:
   12.0.16 is out, worth updating now?



##########
artemis-pom/pom.xml:
##########
@@ -518,13 +518,31 @@
 
          <!-- ## Jetty web Dependencies ##-->
          <dependency>
-            <groupId>org.eclipse.jetty</groupId>
-            <artifactId>jetty-bom</artifactId>
+            <groupId>org.eclipse.jetty.ee8</groupId>
+            <artifactId>jetty-ee8-bom</artifactId>

Review Comment:
   Why switch this rather than add the ee8 bom import in addition to the 
existing one, they appear to be additive? If leaving the existing bom import, 
the 3 new additions below shouldnt be needed?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org
For additional commands, e-mail: gitbox-h...@activemq.apache.org
For further information, visit: https://activemq.apache.org/contact


Reply via email to