From ad9e2d6c53a66646862b4b03f9c27a47b47fe5a4 Mon Sep 17 00:00:00 2001
From: Rob Vesse <rob.vesse@telicent.io>
Date: Thu, 9 Jan 2025 15:05:18 +0000
Subject: [PATCH] Restore wrongly scoped Jersey related dependencies

Even with 0.26.0 release was still getting strange runtime errors in
downstream repositories.  On further investigation realised this was
become some dependencies has been erroneously moved to test scope when
they were in fact required for servers to run correctly.  They were
wrongly classified by dependency analysis as unused because they are
not directly used, rather indirectly via Service Loading.

This also explains why the tests here didn't fail as these dependencies
has simpy been moved to test scope so they were present for the tests
and allowed those to pass successfully.
---
 jaxrs-base-server/pom.xml | 55 ++++++++++++++++++---------------------
 1 file changed, 25 insertions(+), 30 deletions(-)

diff --git a/jaxrs-base-server/pom.xml b/jaxrs-base-server/pom.xml
index 5d7ab0a..ecd49cd 100644
--- a/jaxrs-base-server/pom.xml
+++ b/jaxrs-base-server/pom.xml
@@ -36,6 +36,16 @@
             <artifactId>caffeine</artifactId>
         </dependency>
 
+        <dependency>
+            <groupId>com.fasterxml.jackson.core</groupId>
+            <artifactId>jackson-databind</artifactId>
+        </dependency>
+
+        <dependency>
+            <groupId>com.fasterxml.jackson.core</groupId>
+            <artifactId>jackson-core</artifactId>
+        </dependency>
+
         <dependency>
             <groupId>org.glassfish.grizzly</groupId>
             <artifactId>grizzly-framework</artifactId>
@@ -86,6 +96,16 @@
             <artifactId>grizzly-http</artifactId>
         </dependency>
 
+        <dependency>
+            <groupId>org.glassfish.jersey.inject</groupId>
+            <artifactId>jersey-hk2</artifactId>
+        </dependency>
+
+        <dependency>
+            <groupId>org.glassfish.jersey.media</groupId>
+            <artifactId>jersey-media-json-jackson</artifactId>
+        </dependency>
+
         <dependency>
             <groupId>io.telicent.public</groupId>
             <artifactId>jwt-servlet-auth-core</artifactId>
@@ -131,6 +151,11 @@
             <artifactId>jwt-servlet-auth-jaxrs3</artifactId>
         </dependency>
 
+        <dependency>
+            <groupId>io.telicent.public</groupId>
+            <artifactId>jwt-servlet-auth-aws</artifactId>
+        </dependency>
+
         <dependency>
             <groupId>io.telicent.smart-caches</groupId>
             <artifactId>jwt-auth-common</artifactId>
@@ -173,12 +198,6 @@
             <scope>test</scope>
         </dependency>
 
-        <dependency>
-            <groupId>com.fasterxml.jackson.core</groupId>
-            <artifactId>jackson-databind</artifactId>
-            <scope>test</scope>
-        </dependency>
-
         <dependency>
             <groupId>io.jsonwebtoken</groupId>
             <artifactId>jjwt-jackson</artifactId>
@@ -186,12 +205,6 @@
             <scope>test</scope>
         </dependency>
 
-        <dependency>
-            <groupId>com.fasterxml.jackson.core</groupId>
-            <artifactId>jackson-core</artifactId>
-            <scope>test</scope>
-        </dependency>
-
         <dependency>
             <groupId>io.telicent.public</groupId>
             <artifactId>jwt-servlet-auth-core</artifactId>
@@ -200,24 +213,6 @@
             <scope>test</scope>
         </dependency>
 
-        <dependency>
-            <groupId>io.telicent.public</groupId>
-            <artifactId>jwt-servlet-auth-aws</artifactId>
-            <scope>test</scope>
-        </dependency>
-
-        <dependency>
-            <groupId>org.glassfish.jersey.inject</groupId>
-            <artifactId>jersey-hk2</artifactId>
-            <scope>test</scope>
-        </dependency>
-
-        <dependency>
-            <groupId>org.glassfish.jersey.media</groupId>
-            <artifactId>jersey-media-json-jackson</artifactId>
-            <scope>test</scope>
-        </dependency>
-
     </dependencies>
 
     <build>