From 52eddd13ba822553590de10c3cc4bccf97c82f73 Mon Sep 17 00:00:00 2001
From: Gino Augustine <ginoaugustine@gmail.com>
Date: Tue, 26 Sep 2023 20:56:55 +0530
Subject: [PATCH 01/12] jdk17 support

---
 .github/workflows/build.yml                   |  6 ++---
 Dockerfile                                    |  4 ++--
 docker/README.md                              |  2 +-
 opengrok-indexer/pom.xml                      |  6 ++++-
 .../indexer/framework/PluginFramework.java    |  4 +---
 .../logger/formatter/LogFormatter.java        |  2 +-
 opengrok-web/pom.xml                          |  2 +-
 plugins/pom.xml                               |  2 +-
 pom.xml                                       | 23 +++----------------
 suggester/pom.xml                             |  8 ++++++-
 10 files changed, 25 insertions(+), 34 deletions(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 7a2e794f635..7fb209a662b 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -11,7 +11,7 @@ on:
 
 jobs:
   build:
-    name: ${{ matrix.os }} with Java 11
+    name: ${{ matrix.os }} with Java 17
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false
@@ -22,10 +22,10 @@ jobs:
       uses: actions/checkout@v2
       with:
         fetch-depth: 0
-    - name: Set up JDK 11
+    - name: Set up JDK 17
       uses: actions/setup-java@v1
       with:
-        java-version: 11
+        java-version: 17
     - name: Cache Maven packages
       uses: actions/cache@v2
       with:
diff --git a/Dockerfile b/Dockerfile
index bfc58b82a82..4a078c89f13 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -4,7 +4,7 @@
 FROM ubuntu:jammy as build
 
 # hadolint ignore=DL3008
-RUN apt-get update && apt-get install --no-install-recommends -y openjdk-11-jdk python3 python3-venv && \
+RUN apt-get update && apt-get install --no-install-recommends -y openjdk-17-jdk python3 python3-venv && \
     apt-get clean && \
     rm -rf /var/lib/apt/lists/*
 
@@ -41,7 +41,7 @@ RUN cp `ls -t distribution/target/*.tar.gz | head -1` /opengrok.tar.gz
 # Store the version in a file so that the tools can report it.
 RUN /mvn/mvnw help:evaluate -Dexpression=project.version -q -DforceStdout > /mvn/VERSION
 
-FROM tomcat:10.1-jdk11
+FROM tomcat:10.1.13-jdk17
 LABEL maintainer="https://github.com/oracle/opengrok"
 
 # Add Perforce apt source.
diff --git a/docker/README.md b/docker/README.md
index f2945d9fa82..5e4048dfdb1 100644
--- a/docker/README.md
+++ b/docker/README.md
@@ -26,7 +26,7 @@ image based on the official one.
 ## Additional info about the image
 
 * Tomcat 10
-* JRE 11
+* JRE 17
 * Configurable mirroring/reindexing (default every 10 min)
 
 The mirroring step works by going through all projects and attempting to
diff --git a/opengrok-indexer/pom.xml b/opengrok-indexer/pom.xml
index cce12a1b4b0..16ca0534624 100644
--- a/opengrok-indexer/pom.xml
+++ b/opengrok-indexer/pom.xml
@@ -377,10 +377,14 @@ Portions Copyright (c) 2020-2020, Lubos Kosco <tarzanek@gmail.com>.
                     </excludes>
                     <argLine>
                         @{surefireArgLine}
-                        --illegal-access=permit
+
                     </argLine>
                 </configuration>
             </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-compiler-plugin</artifactId>
+            </plugin>
         </plugins>
     </build>
 
diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/framework/PluginFramework.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/framework/PluginFramework.java
index 63e130e1d72..661d3b6a0b8 100644
--- a/opengrok-indexer/src/main/java/org/opengrok/indexer/framework/PluginFramework.java
+++ b/opengrok-indexer/src/main/java/org/opengrok/indexer/framework/PluginFramework.java
@@ -26,8 +26,6 @@
 import java.io.IOException;
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Modifier;
-import java.security.AccessController;
-import java.security.PrivilegedAction;
 import java.util.Arrays;
 import java.util.Enumeration;
 import java.util.LinkedList;
@@ -382,7 +380,7 @@ public final void reload() {
 
         // trashing out the old instance of the loader enables us
         // to reload the stack at runtime
-        loader = AccessController.doPrivileged((PrivilegedAction<PluginClassLoader>) () -> new PluginClassLoader(pluginDirectory));
+        loader = new PluginClassLoader(pluginDirectory);
 
         // notify the implementing class that the reload is about to begin
         beforeReload();
diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/logger/formatter/LogFormatter.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/logger/formatter/LogFormatter.java
index 6b03dbb6468..f418cf4de84 100644
--- a/opengrok-indexer/src/main/java/org/opengrok/indexer/logger/formatter/LogFormatter.java
+++ b/opengrok-indexer/src/main/java/org/opengrok/indexer/logger/formatter/LogFormatter.java
@@ -93,7 +93,7 @@ public String format(LogRecord record) {
                 record.getSourceClassName(),           //%7
                 record.getSourceMethodName(),          //%8
                 className(record.getSourceClassName()), //%9
-                record.getThreadID(),                  //%10
+                record.getLongThreadID(),                  //%10
                 record.getMessage(),                   //%11
                 version
                 );
diff --git a/opengrok-web/pom.xml b/opengrok-web/pom.xml
index 771e407c0e4..84854f05e8d 100644
--- a/opengrok-web/pom.xml
+++ b/opengrok-web/pom.xml
@@ -253,7 +253,7 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
                     <argLine>
                         @{surefireArgLine}
                         -Djna.nosys=true
-                        --illegal-access=permit
+
                         --add-exports java.base/jdk.internal.ref=ALL-UNNAMED
                         --add-exports java.base/jdk.internal.misc=ALL-UNNAMED
                         --add-exports java.base/sun.nio.ch=ALL-UNNAMED
diff --git a/plugins/pom.xml b/plugins/pom.xml
index 5bdbbc1ee82..74703184e5a 100644
--- a/plugins/pom.xml
+++ b/plugins/pom.xml
@@ -119,7 +119,7 @@ Portions Copyright (c) 2020, Chris Fraire <cfraire@me.com>.
                     </systemPropertyVariables>
                     <argLine>
                         @{surefireArgLine}
-                        --illegal-access=permit
+
                     </argLine>
                 </configuration>
             </plugin>
diff --git a/pom.xml b/pom.xml
index 506866120d9..6aec687b1bb 100644
--- a/pom.xml
+++ b/pom.xml
@@ -62,9 +62,8 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
     <properties>
         <lucene.version>9.8.0</lucene.version>
         <mavenjavadocplugin.version>3.2.0</mavenjavadocplugin.version>
-        <!-- The following changed syntax from e.g. 1.8 to 11. -->
-        <compileSource>11</compileSource>
-        <compileTarget>11</compileTarget>
+        <compileSource>17</compileSource>
+        <compileTarget>17</compileTarget>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <jersey.version>3.1.0</jersey.version>
         <!-- Jackson version needs to match the version of Jackson which Jersey
@@ -231,7 +230,7 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
                           <version>3.0.5</version>
                         </requireMavenVersion>
                         <requireJavaVersion>
-                          <version>11</version>
+                          <version>17</version>
                         </requireJavaVersion>
                       </rules>
                     </configuration>
@@ -411,22 +410,6 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
     </build>
 
     <profiles>
-        <profile>
-            <id>java12plus-11-release</id>
-            <activation>
-                <jdk>[12,20]</jdk>
-            </activation>
-            <build>
-                <plugins>
-                    <plugin>
-                        <artifactId>maven-compiler-plugin</artifactId>
-                        <configuration>
-                            <release>11</release>
-                        </configuration>
-                    </plugin>
-                </plugins>
-            </build>
-        </profile>
         <profile>
             <id>dependency-check</id>
             <build>
diff --git a/suggester/pom.xml b/suggester/pom.xml
index 1a94280e1f0..bf18fb57c99 100644
--- a/suggester/pom.xml
+++ b/suggester/pom.xml
@@ -163,7 +163,13 @@ Portions Copyright (c) 2020, Chris Fraire <cfraire@me.com>.
                     <argLine>
                         @{surefireArgLine}
                         -Djna.nosys=true
-                        --illegal-access=permit
+                        --add-exports jdk.unsupported/sun.misc=ALL-UNNAMED
+                        --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
+                        --add-opens jdk.compiler/com.sun.tools.javac=ALL-UNNAMED
+                        --add-opens java.base/java.lang=ALL-UNNAMED
+                        --add-opens java.base/java.lang.reflect=ALL-UNNAMED
+                        --add-opens java.base/java.io=ALL-UNNAMED
+                        --add-opens java.base/java.util=ALL-UNNAMED
                         --add-exports java.base/jdk.internal.ref=ALL-UNNAMED
                         --add-exports java.base/jdk.internal.misc=ALL-UNNAMED
                         --add-exports java.base/sun.nio.ch=ALL-UNNAMED

From d52542606fca79598f6d34cd614d0b0765030d55 Mon Sep 17 00:00:00 2001
From: Gino Augustine <ginoaugustine@gmail.com>
Date: Tue, 26 Sep 2023 21:35:36 +0530
Subject: [PATCH 02/12]  workflow changes for jdk 17

---
 .github/workflows/apiary.yml          |  2 +-
 .github/workflows/build.yml           |  6 +++---
 .github/workflows/codeql-analysis.yml | 13 ++++++++-----
 .github/workflows/javadoc.yml         | 10 +++++-----
 .github/workflows/release.yml         | 12 ++++++------
 5 files changed, 23 insertions(+), 20 deletions(-)

diff --git a/.github/workflows/apiary.yml b/.github/workflows/apiary.yml
index 16ac153397c..8194980360c 100644
--- a/.github/workflows/apiary.yml
+++ b/.github/workflows/apiary.yml
@@ -16,7 +16,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: Checkout master branch
-      uses: actions/checkout@v2
+      uses: actions/checkout@v4
     - name: Install drafter
       run: npm install drafter
     - name: Build
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 7fb209a662b..f084eefc526 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -19,11 +19,11 @@ jobs:
         os: [ubuntu-latest, macos-latest, windows-latest]
     steps:
     - name: Checkout master branch
-      uses: actions/checkout@v2
+      uses: actions/checkout@v4
       with:
         fetch-depth: 0
     - name: Set up JDK 17
-      uses: actions/setup-java@v1
+      uses: actions/setup-java@v3
       with:
         java-version: 17
     - name: Cache Maven packages
@@ -33,7 +33,7 @@ jobs:
         key: ${{ runner.os }}-m2-${{ hashFiles('**/pom.xml') }}
         restore-keys: ${{ runner.os }}-m2
     - name: Checkout Universal ctags
-      uses: actions/checkout@v2
+      uses: actions/checkout@v4
       with:
         repository: universal-ctags/ctags
         path: ctags
diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
index 0e48d731064..ee8c038a0d4 100644
--- a/.github/workflows/codeql-analysis.yml
+++ b/.github/workflows/codeql-analysis.yml
@@ -21,15 +21,18 @@ jobs:
 
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v2
-
+      uses: actions/checkout@v4
+    - name: Set up JDK 17
+      uses: actions/setup-java@v3
+      with:
+        java-version: 17
     - name: Initialize CodeQL
-      uses: github/codeql-action/init@v1
+      uses: github/codeql-action/init@v2
       with:
         languages: ${{ matrix.language }}
 
     - name: Autobuild
-      uses: github/codeql-action/autobuild@v1
+      uses: github/codeql-action/autobuild@v2
 
     - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@v1
+      uses: github/codeql-action/analyze@v2
diff --git a/.github/workflows/javadoc.yml b/.github/workflows/javadoc.yml
index eb144bbd4e9..86f8554fd9d 100644
--- a/.github/workflows/javadoc.yml
+++ b/.github/workflows/javadoc.yml
@@ -17,11 +17,11 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: Checkout master branch
-      uses: actions/checkout@v2
-    - name: Set up JDK 11
-      uses: actions/setup-java@v1
+      uses: actions/checkout@v4
+    - name: Set up JDK 17
+      uses: actions/setup-java@v3@v1
       with:
-        java-version: 11
+        java-version: 17
     - name: Cache Maven packages
       uses: actions/cache@v2
       with:
@@ -30,7 +30,7 @@ jobs:
         restore-keys: ${{ runner.os }}-m2
     - name: Checkout gh-pages branch
       if: github.repository == 'oracle/opengrok'
-      uses: actions/checkout@v2
+      uses: actions/checkout@v4
       with:
         ref: gh-pages
         path: gh-pages
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
index 35f3a0e0d77..55374e48f2e 100644
--- a/.github/workflows/release.yml
+++ b/.github/workflows/release.yml
@@ -14,7 +14,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout master branch
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
       - name: Get the tag name
         id: get_tag
         env:
@@ -25,11 +25,11 @@ jobs:
     needs: get_tag
     steps:
     - name: Checkout master branch
-      uses: actions/checkout@v2
-    - name: Set up JDK 11
-      uses: actions/setup-java@v1
+      uses: actions/checkout@v4
+    - name: Set up JDK 17
+      uses: actions/setup-java@v3
       with:
-        java-version: 11
+        java-version: 17
     - name: Cache Maven packages
       uses: actions/cache@v2
       with:
@@ -37,7 +37,7 @@ jobs:
         key: ${{ runner.os }}-m2-${{ hashFiles('**/pom.xml') }}
         restore-keys: ${{ runner.os }}-m2
     - name: Checkout Universal ctags
-      uses: actions/checkout@v2
+      uses: actions/checkout@v4
       with:
         repository: universal-ctags/ctags
         path: ctags

From 53be7f3d54fe768eb6fe1b44e0adb932e2566bd5 Mon Sep 17 00:00:00 2001
From: Gino Augustine <ginoaugustine@gmail.com>
Date: Tue, 26 Sep 2023 21:44:36 +0530
Subject: [PATCH 03/12]  Adding jdk distribution name

---
 .github/workflows/build.yml           | 3 ++-
 .github/workflows/codeql-analysis.yml | 3 ++-
 .github/workflows/javadoc.yml         | 5 +++--
 .github/workflows/release.yml         | 3 ++-
 4 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index f084eefc526..ecd445fdfb2 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -25,7 +25,8 @@ jobs:
     - name: Set up JDK 17
       uses: actions/setup-java@v3
       with:
-        java-version: 17
+        distribution: 'oracle'
+        java-version: '17'
     - name: Cache Maven packages
       uses: actions/cache@v2
       with:
diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
index ee8c038a0d4..41be348d4f9 100644
--- a/.github/workflows/codeql-analysis.yml
+++ b/.github/workflows/codeql-analysis.yml
@@ -25,7 +25,8 @@ jobs:
     - name: Set up JDK 17
       uses: actions/setup-java@v3
       with:
-        java-version: 17
+        distribution: 'oracle'
+        java-version: '17'
     - name: Initialize CodeQL
       uses: github/codeql-action/init@v2
       with:
diff --git a/.github/workflows/javadoc.yml b/.github/workflows/javadoc.yml
index 86f8554fd9d..00829dffa81 100644
--- a/.github/workflows/javadoc.yml
+++ b/.github/workflows/javadoc.yml
@@ -19,9 +19,10 @@ jobs:
     - name: Checkout master branch
       uses: actions/checkout@v4
     - name: Set up JDK 17
-      uses: actions/setup-java@v3@v1
+      uses: actions/setup-java@v3
       with:
-        java-version: 17
+        distribution: 'oracle'
+        java-version: '17'
     - name: Cache Maven packages
       uses: actions/cache@v2
       with:
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
index 55374e48f2e..7a0843ce5d3 100644
--- a/.github/workflows/release.yml
+++ b/.github/workflows/release.yml
@@ -29,7 +29,8 @@ jobs:
     - name: Set up JDK 17
       uses: actions/setup-java@v3
       with:
-        java-version: 17
+        distribution: 'oracle'
+        java-version: '17'
     - name: Cache Maven packages
       uses: actions/cache@v2
       with:

From 68906811cf17e9ea5a0b3fb65853c60db818ba98 Mon Sep 17 00:00:00 2001
From: Gino Augustine <ginoaugustine@gmail.com>
Date: Tue, 26 Sep 2023 23:02:56 +0530
Subject: [PATCH 04/12]  Upgrading maven javadoc plugin version

Rebase from master
---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 6aec687b1bb..22e8e97ec00 100644
--- a/pom.xml
+++ b/pom.xml
@@ -61,7 +61,7 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
 
     <properties>
         <lucene.version>9.8.0</lucene.version>
-        <mavenjavadocplugin.version>3.2.0</mavenjavadocplugin.version>
+        <mavenjavadocplugin.version>3.6.0</mavenjavadocplugin.version>
         <compileSource>17</compileSource>
         <compileTarget>17</compileTarget>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>

From 97deaee1ef48791d1d81de8dc0390eb7395251cf Mon Sep 17 00:00:00 2001
From: Gino Augustine <ginoaugustine@gmail.com>
Date: Tue, 26 Sep 2023 23:41:19 +0530
Subject: [PATCH 05/12]  Correct javadoc error

---
 .../org/opengrok/indexer/configuration/RuntimeEnvironment.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/configuration/RuntimeEnvironment.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/configuration/RuntimeEnvironment.java
index 8f9d92c9715..3b748388dfd 100644
--- a/opengrok-indexer/src/main/java/org/opengrok/indexer/configuration/RuntimeEnvironment.java
+++ b/opengrok-indexer/src/main/java/org/opengrok/indexer/configuration/RuntimeEnvironment.java
@@ -1849,7 +1849,7 @@ public void applyConfig(String configuration, boolean reindex, CommandTimeoutTyp
      * @param reindex is the message result of reindex
      * @param cmdType command timeout type
      * @see ConfigurationChangedListener
-     * </p>
+     *
      */
     public void applyConfig(Configuration config, boolean reindex, CommandTimeoutType cmdType) {
         setConfiguration(config, cmdType);

From 07fdc2ceef33faf8ff558b79ff36acd3adddd0d5 Mon Sep 17 00:00:00 2001
From: Gino Augustine <ginoaugustine@gmail.com>
Date: Thu, 28 Sep 2023 08:50:20 +0530
Subject: [PATCH 06/12] add chroniclemap jdk17 params

Signed-off-by: Gino Augustine <ginoaugustine@gmail.com>
---
 Dockerfile                                    |  5 +++
 .../logger/formatter/LogFormatter.java        | 35 ++++++++++---------
 opengrok-web/pom.xml                          |  8 ++++-
 3 files changed, 30 insertions(+), 18 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index 4a078c89f13..065c4e68b3d 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -93,6 +93,11 @@ ENV CATALINA_BASE /usr/local/tomcat
 ENV CATALINA_TMPDIR /usr/local/tomcat/temp
 ENV PATH $CATALINA_HOME/bin:$PATH
 ENV CLASSPATH /usr/local/tomcat/bin/bootstrap.jar:/usr/local/tomcat/bin/tomcat-juli.jar
+ENV JAVA_OPTS="--add-exports=java.base/jdk.internal.ref=ALL-UNNAMED --add-exports=java.base/sun.nio.ch=ALL-UNNAMED \
+--add-exports=jdk.unsupported/sun.misc=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED \
+--add-opens=jdk.compiler/com.sun.tools.javac=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED \
+--add-opens=java.base/java.lang.reflect=ALL-UNNAMED --add-opens=java.base/java.io=ALL-UNNAMED \
+--add-opens=java.base/java.util=ALL-UNNAMED"
 
 # disable all file logging
 COPY docker/logging.properties /usr/local/tomcat/conf/logging.properties
diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/logger/formatter/LogFormatter.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/logger/formatter/LogFormatter.java
index f418cf4de84..8e7cfb2cc72 100644
--- a/opengrok-indexer/src/main/java/org/opengrok/indexer/logger/formatter/LogFormatter.java
+++ b/opengrok-indexer/src/main/java/org/opengrok/indexer/logger/formatter/LogFormatter.java
@@ -61,24 +61,25 @@ public LogFormatter(String format, String version) {
     }
 
     @Override
-    public String format(LogRecord record) {
-        Date dat = new Date(record.getMillis());
+    @SuppressWarnings("deprecation")
+    public String format(LogRecord logRecord) {
+        Date dat = new Date(logRecord.getMillis());
         StringBuilder source = new StringBuilder();
-        if (record.getSourceClassName() != null) {
-            source.append(record.getSourceClassName());
-            if (record.getSourceMethodName() != null) {
-                source.append(' ').append(record.getSourceMethodName());
+        if (logRecord.getSourceClassName() != null) {
+            source.append(logRecord.getSourceClassName());
+            if (logRecord.getSourceMethodName() != null) {
+                source.append(' ').append(logRecord.getSourceMethodName());
             }
         } else {
-            source.append(record.getLoggerName());
+            source.append(logRecord.getLoggerName());
         }
 
         StringBuilder throwable = new StringBuilder();
-        if (record.getThrown() != null) {
+        if (logRecord.getThrown() != null) {
             StringWriter sw = new StringWriter();
             PrintWriter pw = new PrintWriter(sw);
             pw.println();
-            record.getThrown().printStackTrace(pw);
+            logRecord.getThrown().printStackTrace(pw);
             pw.close();
             throwable.append(sw.toString());
         }
@@ -86,15 +87,15 @@ public String format(LogRecord record) {
         return String.format(format,
                 dat,                                   //%1
                 source.toString(),                     //%2
-                record.getLoggerName(),                //%3
-                record.getLevel().getLocalizedName(),  //%4
-                formatMessage(record),                 //%5
+                logRecord.getLoggerName(),                //%3
+                logRecord.getLevel().getLocalizedName(),  //%4
+                formatMessage(logRecord),                 //%5
                 throwable,                             //%6 (till here the same as JDK7's SimpleFormatter)
-                record.getSourceClassName(),           //%7
-                record.getSourceMethodName(),          //%8
-                className(record.getSourceClassName()), //%9
-                record.getLongThreadID(),                  //%10
-                record.getMessage(),                   //%11
+                logRecord.getSourceClassName(),           //%7
+                logRecord.getSourceMethodName(),          //%8
+                className(logRecord.getSourceClassName()), //%9
+                logRecord.getThreadID(),                  //%10
+                logRecord.getMessage(),                   //%11
                 version
                 );
     }
diff --git a/opengrok-web/pom.xml b/opengrok-web/pom.xml
index 84854f05e8d..8532443d55c 100644
--- a/opengrok-web/pom.xml
+++ b/opengrok-web/pom.xml
@@ -253,7 +253,13 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
                     <argLine>
                         @{surefireArgLine}
                         -Djna.nosys=true
-
+                        --add-exports jdk.unsupported/sun.misc=ALL-UNNAMED
+                        --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
+                        --add-opens jdk.compiler/com.sun.tools.javac=ALL-UNNAMED
+                        --add-opens java.base/java.lang=ALL-UNNAMED
+                        --add-opens java.base/java.lang.reflect=ALL-UNNAMED
+                        --add-opens java.base/java.io=ALL-UNNAMED
+                        --add-opens java.base/java.util=ALL-UNNAMED
                         --add-exports java.base/jdk.internal.ref=ALL-UNNAMED
                         --add-exports java.base/jdk.internal.misc=ALL-UNNAMED
                         --add-exports java.base/sun.nio.ch=ALL-UNNAMED

From 7e115c06287432a4023c5a7bb529f5c8ba7e5dc5 Mon Sep 17 00:00:00 2001
From: Gino Augustine <ginoaugustine@gmail.com>
Date: Thu, 28 Sep 2023 22:41:47 +0530
Subject: [PATCH 07/12]  revert back jdk version to 11

Signed-off-by: Gino Augustine <ginoaugustine@gmail.com>
---
 pom.xml | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/pom.xml b/pom.xml
index 22e8e97ec00..a252cde1e37 100644
--- a/pom.xml
+++ b/pom.xml
@@ -62,8 +62,7 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
     <properties>
         <lucene.version>9.8.0</lucene.version>
         <mavenjavadocplugin.version>3.6.0</mavenjavadocplugin.version>
-        <compileSource>17</compileSource>
-        <compileTarget>17</compileTarget>
+        <java.version>11</java.version>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <jersey.version>3.1.0</jersey.version>
         <!-- Jackson version needs to match the version of Jackson which Jersey
@@ -230,7 +229,7 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
                           <version>3.0.5</version>
                         </requireMavenVersion>
                         <requireJavaVersion>
-                          <version>17</version>
+                          <version>${java.version}</version>
                         </requireJavaVersion>
                       </rules>
                     </configuration>
@@ -250,8 +249,7 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-compiler-plugin</artifactId>
                 <configuration>
-                    <source>${compileSource}</source>
-                    <target>${compileTarget}</target>
+                    <release>${java.version}</release>
                     <encoding>${project.build.sourceEncoding}</encoding>
                     <compilerArgs>
                       <arg>-Xlint:all</arg>

From f6db9c994ba600936ed11b464177dce8c4c33c16 Mon Sep 17 00:00:00 2001
From: Gino Augustine <ginoaugustine@gmail.com>
Date: Tue, 3 Oct 2023 09:17:54 +0530
Subject: [PATCH 08/12] Reverting security manager removal

Signed-off-by: Gino Augustine <ginoaugustine@gmail.com>
---
 .github/workflows/build.yml                                   | 2 +-
 .github/workflows/docker.yml                                  | 2 +-
 .github/workflows/javadoc.yml                                 | 2 +-
 .github/workflows/release.yml                                 | 2 +-
 .../java/org/opengrok/indexer/framework/PluginFramework.java  | 4 +++-
 pom.xml                                                       | 2 +-
 6 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index ecd445fdfb2..5e86d2b9632 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -28,7 +28,7 @@ jobs:
         distribution: 'oracle'
         java-version: '17'
     - name: Cache Maven packages
-      uses: actions/cache@v2
+      uses: actions/cache@v3
       with:
         path: ~/.m2
         key: ${{ runner.os }}-m2-${{ hashFiles('**/pom.xml') }}
diff --git a/.github/workflows/docker.yml b/.github/workflows/docker.yml
index 71d85168227..37d83e15b16 100644
--- a/.github/workflows/docker.yml
+++ b/.github/workflows/docker.yml
@@ -7,7 +7,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: Checkout master branch
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
     - uses: actions/setup-python@v4
       with:
         python-version: '3.10'
diff --git a/.github/workflows/javadoc.yml b/.github/workflows/javadoc.yml
index 00829dffa81..c9e6365cd87 100644
--- a/.github/workflows/javadoc.yml
+++ b/.github/workflows/javadoc.yml
@@ -24,7 +24,7 @@ jobs:
         distribution: 'oracle'
         java-version: '17'
     - name: Cache Maven packages
-      uses: actions/cache@v2
+      uses: actions/cache@v3
       with:
         path: ~/.m2
         key: ${{ runner.os }}-m2-${{ hashFiles('**/pom.xml') }}
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
index 7a0843ce5d3..bab913448c8 100644
--- a/.github/workflows/release.yml
+++ b/.github/workflows/release.yml
@@ -32,7 +32,7 @@ jobs:
         distribution: 'oracle'
         java-version: '17'
     - name: Cache Maven packages
-      uses: actions/cache@v2
+      uses: actions/cache@v3
       with:
         path: ~/.m2
         key: ${{ runner.os }}-m2-${{ hashFiles('**/pom.xml') }}
diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/framework/PluginFramework.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/framework/PluginFramework.java
index 661d3b6a0b8..8d80db679a9 100644
--- a/opengrok-indexer/src/main/java/org/opengrok/indexer/framework/PluginFramework.java
+++ b/opengrok-indexer/src/main/java/org/opengrok/indexer/framework/PluginFramework.java
@@ -380,7 +380,9 @@ public final void reload() {
 
         // trashing out the old instance of the loader enables us
         // to reload the stack at runtime
-        loader = new PluginClassLoader(pluginDirectory);
+        loader = java.security.AccessController.doPrivileged(
+                (java.security.PrivilegedAction<PluginClassLoader>) () -> new PluginClassLoader(pluginDirectory)
+        );
 
         // notify the implementing class that the reload is about to begin
         beforeReload();
diff --git a/pom.xml b/pom.xml
index a252cde1e37..d4c4fad82e8 100644
--- a/pom.xml
+++ b/pom.xml
@@ -485,7 +485,7 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
                     <rulesets>
                         <ruleset>dev/pmd_ruleset.xml</ruleset>
                     </rulesets>
-                    <targetJdk>${compileTarget}</targetJdk>
+                    <targetJdk>${java.version}</targetJdk>
                     <aggregate>true</aggregate>
                 </configuration>
             </plugin>

From 56cd6251cc150a5193d2cd0cf35df342da6cd4c0 Mon Sep 17 00:00:00 2001
From: Gino Augustine <ginoaugustine@gmail.com>
Date: Tue, 3 Oct 2023 15:21:31 +0530
Subject: [PATCH 09/12] Removing unused supress warning

Signed-off-by: Gino Augustine <ginoaugustine@gmail.com>
---
 .../org/opengrok/indexer/framework/PluginFramework.java    | 4 +---
 .../indexer/index/DefaultIndexChangedListener.java         | 1 -
 .../src/main/java/org/opengrok/indexer/index/Indexer.java  | 3 +--
 .../java/org/opengrok/indexer/web/messages/Message.java    | 1 -
 .../opengrok/indexer/history/BazaarHistoryParserTest.java  | 1 -
 opengrok-web/pom.xml                                       | 5 ++---
 .../web/api/v1/suggester/query/SuggesterQueryParser.java   | 1 -
 .../web/api/v1/controller/SuggesterControllerTest.java     | 3 ---
 pom.xml                                                    | 2 +-
 suggester/pom.xml                                          | 7 +++----
 .../popular/impl/chronicle/BytesRefSizedReader.java        | 1 -
 11 files changed, 8 insertions(+), 21 deletions(-)

diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/framework/PluginFramework.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/framework/PluginFramework.java
index 8d80db679a9..661d3b6a0b8 100644
--- a/opengrok-indexer/src/main/java/org/opengrok/indexer/framework/PluginFramework.java
+++ b/opengrok-indexer/src/main/java/org/opengrok/indexer/framework/PluginFramework.java
@@ -380,9 +380,7 @@ public final void reload() {
 
         // trashing out the old instance of the loader enables us
         // to reload the stack at runtime
-        loader = java.security.AccessController.doPrivileged(
-                (java.security.PrivilegedAction<PluginClassLoader>) () -> new PluginClassLoader(pluginDirectory)
-        );
+        loader = new PluginClassLoader(pluginDirectory);
 
         // notify the implementing class that the reload is about to begin
         beforeReload();
diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/index/DefaultIndexChangedListener.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/index/DefaultIndexChangedListener.java
index b55f01fa77a..4bb1282decb 100644
--- a/opengrok-indexer/src/main/java/org/opengrok/indexer/index/DefaultIndexChangedListener.java
+++ b/opengrok-indexer/src/main/java/org/opengrok/indexer/index/DefaultIndexChangedListener.java
@@ -37,7 +37,6 @@
  *
  * @author Trond Norbye
  */
-@SuppressWarnings("PMD.SystemPrintln")
 public class DefaultIndexChangedListener implements IndexChangedListener {
 
     private static final Logger LOGGER = LoggerFactory.getLogger(DefaultIndexChangedListener.class);
diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/index/Indexer.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/index/Indexer.java
index 3d989cab7da..ab80682049e 100644
--- a/opengrok-indexer/src/main/java/org/opengrok/indexer/index/Indexer.java
+++ b/opengrok-indexer/src/main/java/org/opengrok/indexer/index/Indexer.java
@@ -96,7 +96,7 @@
  * Windows already supports opening {@code /var/opengrok} as {@code C:\var\opengrok}
  * </p>
  */
-@SuppressWarnings({"PMD.AvoidPrintStackTrace", "PMD.SystemPrintln"})
+@SuppressWarnings({"PMD.AvoidPrintStackTrace", "PMD.SystemPrintln", "java:S106"})
 public final class Indexer {
 
     private static final Logger LOGGER = LoggerFactory.getLogger(Indexer.class);
@@ -162,7 +162,6 @@ public static Indexer getInstance() {
     @SuppressWarnings("PMD.UseStringBufferForStringAppends")
     public static void main(String[] argv) {
         Statistics stats = new Statistics(); //this won't count JVM creation though
-        boolean update = true;
 
         Executor.registerErrorHandler();
         List<String> subFiles = RuntimeEnvironment.getInstance().getSubFiles();
diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/web/messages/Message.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/web/messages/Message.java
index ca7a4ce2bbf..440862ce33a 100644
--- a/opengrok-indexer/src/main/java/org/opengrok/indexer/web/messages/Message.java
+++ b/opengrok-indexer/src/main/java/org/opengrok/indexer/web/messages/Message.java
@@ -80,7 +80,6 @@ public String toString() {
             return messageLevelString;
         }
 
-        @SuppressWarnings("rawtypes")
         public static final Comparator<MessageLevel> VALUE_COMPARATOR = Comparator.comparingInt(Enum::ordinal);
     }
 
diff --git a/opengrok-indexer/src/test/java/org/opengrok/indexer/history/BazaarHistoryParserTest.java b/opengrok-indexer/src/test/java/org/opengrok/indexer/history/BazaarHistoryParserTest.java
index 059011ce316..cfb4174dbf2 100644
--- a/opengrok-indexer/src/test/java/org/opengrok/indexer/history/BazaarHistoryParserTest.java
+++ b/opengrok-indexer/src/test/java/org/opengrok/indexer/history/BazaarHistoryParserTest.java
@@ -143,7 +143,6 @@ public void parseLogNoFile() throws Exception {
     }
 
     @Test
-    @SuppressWarnings("unchecked")
     public void parseLogDirectory() throws Exception {
         String revId1 = "1234";
         String author1 = "username@example.com";
diff --git a/opengrok-web/pom.xml b/opengrok-web/pom.xml
index 8532443d55c..1a10d1ce498 100644
--- a/opengrok-web/pom.xml
+++ b/opengrok-web/pom.xml
@@ -253,6 +253,8 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
                     <argLine>
                         @{surefireArgLine}
                         -Djna.nosys=true
+                        --add-exports java.base/jdk.internal.ref=ALL-UNNAMED
+                        --add-exports java.base/sun.nio.ch=ALL-UNNAMED
                         --add-exports jdk.unsupported/sun.misc=ALL-UNNAMED
                         --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
                         --add-opens jdk.compiler/com.sun.tools.javac=ALL-UNNAMED
@@ -260,9 +262,6 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
                         --add-opens java.base/java.lang.reflect=ALL-UNNAMED
                         --add-opens java.base/java.io=ALL-UNNAMED
                         --add-opens java.base/java.util=ALL-UNNAMED
-                        --add-exports java.base/jdk.internal.ref=ALL-UNNAMED
-                        --add-exports java.base/jdk.internal.misc=ALL-UNNAMED
-                        --add-exports java.base/sun.nio.ch=ALL-UNNAMED
                     </argLine>
                 </configuration>
             </plugin>
diff --git a/opengrok-web/src/main/java/org/opengrok/web/api/v1/suggester/query/SuggesterQueryParser.java b/opengrok-web/src/main/java/org/opengrok/web/api/v1/suggester/query/SuggesterQueryParser.java
index 718fbeb3b46..ba9c34ea0ce 100644
--- a/opengrok-web/src/main/java/org/opengrok/web/api/v1/suggester/query/SuggesterQueryParser.java
+++ b/opengrok-web/src/main/java/org/opengrok/web/api/v1/suggester/query/SuggesterQueryParser.java
@@ -237,7 +237,6 @@ protected Query newFuzzyQuery(final Term term, final float minimumSimilarity, fi
                 replaceIdentifier(term.field(), term.text() + "~" + ((int) minimumSimilarity));
             }
 
-            @SuppressWarnings("deprecation")
             int numEdits = FuzzyQuery.floatToEdits(minimumSimilarity,
                     newTerm.text().codePointCount(0, newTerm.text().length()));
 
diff --git a/opengrok-web/src/test/java/org/opengrok/web/api/v1/controller/SuggesterControllerTest.java b/opengrok-web/src/test/java/org/opengrok/web/api/v1/controller/SuggesterControllerTest.java
index ca893cca5d9..574af6563d8 100644
--- a/opengrok-web/src/test/java/org/opengrok/web/api/v1/controller/SuggesterControllerTest.java
+++ b/opengrok-web/src/test/java/org/opengrok/web/api/v1/controller/SuggesterControllerTest.java
@@ -596,7 +596,6 @@ void testAllowComplexQueries() {
     }
 
     @Test
-    @SuppressWarnings("unchecked") // for contains
     void testGetPopularityDataSimple() {
         assertTrue(SuggesterServiceImpl.getInstance().increaseSearchCount("rust",
                 new Term(QueryBuilder.FULL, "main"), 10, true));
@@ -611,7 +610,6 @@ void testGetPopularityDataSimple() {
     }
 
     @Test
-    @SuppressWarnings("unchecked") // for contains
     void testGetPopularityDataAll() {
         assertTrue(SuggesterServiceImpl.getInstance().increaseSearchCount("csharp",
                 new Term(QueryBuilder.FULL, "mynamespace"), 10, true));
@@ -630,7 +628,6 @@ void testGetPopularityDataAll() {
     }
 
     @Test
-    @SuppressWarnings("unchecked") // for contains
     void testGetPopularityDataDifferentField() {
         assertTrue(SuggesterServiceImpl.getInstance().increaseSearchCount("swift",
                 new Term(QueryBuilder.FULL, "print"), 10, true));
diff --git a/pom.xml b/pom.xml
index d4c4fad82e8..33decdf2983 100644
--- a/pom.xml
+++ b/pom.xml
@@ -62,7 +62,7 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
     <properties>
         <lucene.version>9.8.0</lucene.version>
         <mavenjavadocplugin.version>3.6.0</mavenjavadocplugin.version>
-        <java.version>11</java.version>
+        <java.version>17</java.version>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <jersey.version>3.1.0</jersey.version>
         <!-- Jackson version needs to match the version of Jackson which Jersey
diff --git a/suggester/pom.xml b/suggester/pom.xml
index bf18fb57c99..cb9da725167 100644
--- a/suggester/pom.xml
+++ b/suggester/pom.xml
@@ -37,7 +37,7 @@ Portions Copyright (c) 2020, Chris Fraire <cfraire@me.com>.
     <name>OpenGrok Suggester</name>
 
     <properties>
-        <jdkArgs></jdkArgs>
+        <jdkArgs/>
     </properties>
 
     <dependencies>
@@ -163,6 +163,8 @@ Portions Copyright (c) 2020, Chris Fraire <cfraire@me.com>.
                     <argLine>
                         @{surefireArgLine}
                         -Djna.nosys=true
+                        --add-exports java.base/jdk.internal.ref=ALL-UNNAMED
+                        --add-exports java.base/sun.nio.ch=ALL-UNNAMED
                         --add-exports jdk.unsupported/sun.misc=ALL-UNNAMED
                         --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
                         --add-opens jdk.compiler/com.sun.tools.javac=ALL-UNNAMED
@@ -170,9 +172,6 @@ Portions Copyright (c) 2020, Chris Fraire <cfraire@me.com>.
                         --add-opens java.base/java.lang.reflect=ALL-UNNAMED
                         --add-opens java.base/java.io=ALL-UNNAMED
                         --add-opens java.base/java.util=ALL-UNNAMED
-                        --add-exports java.base/jdk.internal.ref=ALL-UNNAMED
-                        --add-exports java.base/jdk.internal.misc=ALL-UNNAMED
-                        --add-exports java.base/sun.nio.ch=ALL-UNNAMED
                     </argLine>
                 </configuration>
             </plugin>
diff --git a/suggester/src/main/java/org/opengrok/suggest/popular/impl/chronicle/BytesRefSizedReader.java b/suggester/src/main/java/org/opengrok/suggest/popular/impl/chronicle/BytesRefSizedReader.java
index 01154061281..730ef0c29de 100644
--- a/suggester/src/main/java/org/opengrok/suggest/popular/impl/chronicle/BytesRefSizedReader.java
+++ b/suggester/src/main/java/org/opengrok/suggest/popular/impl/chronicle/BytesRefSizedReader.java
@@ -46,7 +46,6 @@ private BytesRefSizedReader() {
 
     @NotNull
     @Override
-    @SuppressWarnings("rawtypes")
     public BytesRef read(Bytes in, long size, @Nullable BytesRef using) {
         if (size < 0L || size > (long) Integer.MAX_VALUE) {
             throw new IORuntimeException("byte[] size should be non-negative int, " +

From eea479f80f2a06e4078b218e6b31cd741a463515 Mon Sep 17 00:00:00 2001
From: Gino Augustine <ginoaugustine@gmail.com>
Date: Tue, 3 Oct 2023 17:30:13 +0530
Subject: [PATCH 10/12] upgrade third party dependencies

Signed-off-by: Gino Augustine <ginoaugustine@gmail.com>
---
 pom.xml | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/pom.xml b/pom.xml
index 33decdf2983..b5f9085436e 100644
--- a/pom.xml
+++ b/pom.xml
@@ -64,18 +64,18 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
         <mavenjavadocplugin.version>3.6.0</mavenjavadocplugin.version>
         <java.version>17</java.version>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-        <jersey.version>3.1.0</jersey.version>
+        <jersey.version>3.1.3</jersey.version>
         <!-- Jackson version needs to match the version of Jackson which Jersey
         (notably the jersey-media-json-jackson artifact used in opengrok-web)
         depends on or otherwise weird things will happen. -->
-        <jackson.version>2.14.1</jackson.version>
-        <junit.version>5.7.2</junit.version>
+        <jackson.version>2.15.2</jackson.version>
+        <junit.version>5.10.0</junit.version>
         <hamcrest.version>2.2</hamcrest.version>
         <maven-surefire.version>3.0.0-M5</maven-surefire.version>
-        <apache-commons-lang3.version>3.12.0</apache-commons-lang3.version>
-        <micrometer.version>1.8.2</micrometer.version>
+        <apache-commons-lang3.version>3.13.0</apache-commons-lang3.version>
+        <micrometer.version>1.11.4</micrometer.version>
         <mockito.version>3.12.4</mockito.version>
-        <commons-io.version>1.3.2</commons-io.version>
+        <commons-io.version>2.14.0</commons-io.version>
     </properties>
 
     <dependencyManagement>

From 76d03e3969d5fc0863458db30d37f7afef4d9bfd Mon Sep 17 00:00:00 2001
From: Gino Augustine <ginoaugustine@gmail.com>
Date: Tue, 3 Oct 2023 18:04:19 +0530
Subject: [PATCH 11/12] Revert Minimum Build Version to JDK11

Signed-off-by: Gino Augustine <ginoaugustine@gmail.com>
---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index b5f9085436e..5f2bae474d5 100644
--- a/pom.xml
+++ b/pom.xml
@@ -62,7 +62,7 @@ Portions Copyright (c) 2018, 2020, Chris Fraire <cfraire@me.com>.
     <properties>
         <lucene.version>9.8.0</lucene.version>
         <mavenjavadocplugin.version>3.6.0</mavenjavadocplugin.version>
-        <java.version>17</java.version>
+        <java.version>11</java.version>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <jersey.version>3.1.3</jersey.version>
         <!-- Jackson version needs to match the version of Jackson which Jersey

From e95f24d335827cabdda7235df35afab05eb89d90 Mon Sep 17 00:00:00 2001
From: Gino Augustine <ginoaugustine@gmail.com>
Date: Fri, 6 Oct 2023 19:25:12 +0530
Subject: [PATCH 12/12] added back suppress warnings for rawtypes

Signed-off-by: Gino Augustine <ginoaugustine@gmail.com>
---
 .../suggest/popular/impl/chronicle/BytesRefSizedReader.java  | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/suggester/src/main/java/org/opengrok/suggest/popular/impl/chronicle/BytesRefSizedReader.java b/suggester/src/main/java/org/opengrok/suggest/popular/impl/chronicle/BytesRefSizedReader.java
index 730ef0c29de..63baad2404d 100644
--- a/suggester/src/main/java/org/opengrok/suggest/popular/impl/chronicle/BytesRefSizedReader.java
+++ b/suggester/src/main/java/org/opengrok/suggest/popular/impl/chronicle/BytesRefSizedReader.java
@@ -35,7 +35,7 @@
 
 /**
  * {@link BytesRef} data serializer for {@link net.openhft.chronicle.map.ChronicleMap}.
- * Modified from https://github.com/OpenHFT/Chronicle-Map/blob/master/docs/CM_Tutorial_DataAccess.adoc
+ * Modified from <a href="https://github.com/OpenHFT/Chronicle-Map/blob/master/docs/CM_Tutorial_DataAccess.adoc">...</a>
  */
 public class BytesRefSizedReader implements SizedReader<BytesRef>, Marshallable, ReadResolvable<BytesRefSizedReader> {
 
@@ -46,8 +46,9 @@ private BytesRefSizedReader() {
 
     @NotNull
     @Override
+    @SuppressWarnings({"rawtypes"})
     public BytesRef read(Bytes in, long size, @Nullable BytesRef using) {
-        if (size < 0L || size > (long) Integer.MAX_VALUE) {
+        if (size < 0L || size > Integer.MAX_VALUE) {
             throw new IORuntimeException("byte[] size should be non-negative int, " +
                     size + " given. Memory corruption?");
         }