Modernize to Jenkins 2.479 and Jakarta EE 9

* Adapter methods are added for old overrides.
* Switch to JUnit 5 for tests

Author: mtughan

.gitignore

@@ -7,3 +7,4 @@ mvn.out*
 .settings/
 .repository/
 .settings.xml
+.idea/

pom.xml

@@ -5,7 +5,7 @@
   <parent>
     <groupId>org.jenkins-ci.plugins</groupId>
     <artifactId>plugin</artifactId>
-    <version>4.86</version>
+    <version>5.6</version>
     <relativePath />
   </parent>
 
@@ -34,8 +34,8 @@
   <properties>
     <changelist>999999-SNAPSHOT</changelist>
     <!-- https://www.jenkins.io/doc/developer/plugin-development/choosing-jenkins-baseline/ -->
-    <jenkins.baseline>2.440</jenkins.baseline>
-    <jenkins.version>${jenkins.baseline}.3</jenkins.version>
+    <jenkins.baseline>2.479</jenkins.baseline>
+    <jenkins.version>${jenkins.baseline}.1</jenkins.version>
     <gitHubRepo>jenkinsci/${project.artifactId}-plugin</gitHubRepo>
   </properties>
 
@@ -44,7 +44,7 @@
       <dependency>
         <groupId>io.jenkins.tools.bom</groupId>
         <artifactId>bom-${jenkins.baseline}.x</artifactId>
-        <version>3234.v5ca_5154341ef</version>
+        <version>4023.va_eeb_b_4e45f07</version>
         <type>pom</type>
         <scope>import</scope>
       </dependency>
@@ -68,7 +68,7 @@
     <dependency>
       <groupId>org.eclipse.jgit</groupId>
       <artifactId>org.eclipse.jgit.ssh.apache</artifactId>
-      <version>6.9.0.202403050737-r</version>
+      <version>7.0.0.202409031743-r</version>
       <scope>test</scope>
     </dependency>
     <dependency>

src/main/java/org/jenkinsci/plugins/gitserver/CSRFExclusionImpl.java

@@ -2,18 +2,18 @@
 
 import hudson.Extension;
 import hudson.security.csrf.CrumbExclusion;
+import jakarta.servlet.FilterChain;
+import jakarta.servlet.ReadListener;
+import jakarta.servlet.ServletException;
+import jakarta.servlet.ServletInputStream;
+import jakarta.servlet.http.HttpServletRequest;
+import jakarta.servlet.http.HttpServletRequestWrapper;
+import jakarta.servlet.http.HttpServletResponse;
 import java.io.IOException;
 import java.util.Collections;
 import java.util.Enumeration;
+import java.util.Locale;
 import java.util.Map;
-import java.util.Vector;
-import javax.servlet.FilterChain;
-import javax.servlet.ReadListener;
-import javax.servlet.ServletException;
-import javax.servlet.ServletInputStream;
-import javax.servlet.http.HttpServletRequest;
-import javax.servlet.http.HttpServletRequestWrapper;
-import javax.servlet.http.HttpServletResponse;
 
 /**
  * CSRF exclusion for git-upload-pack.
@@ -23,6 +23,7 @@
  * because of the dynamic nature of the URL structure, this doesn't guarantee
  * that we have no leak.
  *
+ * <p>
  * So to further protect Jenkins, we pass through a fake {@link HttpServletRequest}
  * that masks the values of the submission.
  *
@@ -39,48 +40,46 @@
  */
 @Extension
 public class CSRFExclusionImpl extends CrumbExclusion {
+    private static final String BOGUS = "bogus";
 
+    @Override
     public boolean process(HttpServletRequest request, HttpServletResponse response, FilterChain chain)
             throws IOException, ServletException {
         if (!"application/x-git-receive-pack-request".equals(request.getHeader("Content-Type"))) return false;
 
-        //        String path = request.getPathInfo();
-        //        if(!path.contains("/repo.git/") || !path.endsWith("/git-receive-pack"))
-        //            return false;
-
         HttpServletRequestWrapper w = new HttpServletRequestWrapper(request) {
             @Override
             public String getQueryString() {
-                return "bogus";
+                return BOGUS;
             }
 
             @Override
             public String getParameter(String name) {
-                return "bogus";
+                return BOGUS;
             }
 
             @Override
-            public Map getParameterMap() {
+            public Map<String, String[]> getParameterMap() {
                 return Collections.emptyMap();
             }
 
             @Override
-            public Enumeration getParameterNames() {
-                return new Vector().elements();
+            public Enumeration<String> getParameterNames() {
+                return Collections.emptyEnumeration();
             }
 
             @Override
             public String[] getParameterValues(String name) {
-                return new String[] {"bogus"};
+                return new String[] {BOGUS};
             }
 
             @Override
             public String getMethod() {
-                return "BOGUS";
+                return BOGUS.toUpperCase(Locale.ROOT);
             }
 
             @Override
-            public ServletInputStream getInputStream() throws IOException {
+            public ServletInputStream getInputStream() {
                 return new ServletInputStream() {
                     @Override
                     public boolean isFinished() {
@@ -93,7 +92,7 @@ public boolean isReady() {
                     }
 
                     @Override
-                    public int read() throws IOException {
+                    public int read() {
                         return -1;
                     }
 

src/main/java/org/jenkinsci/plugins/gitserver/ChannelTransport.java

@@ -53,6 +53,7 @@ public FetchConnection openFetch() throws NotSupportedException, TransportExcept
         } catch (IOException e) {
             throw new TransportException("Failed to open a fetch connection", e);
         } catch (InterruptedException e) {
+            Thread.currentThread().interrupt();
             throw new TransportException("Failed to open a fetch connection", e);
         }
 
@@ -74,6 +75,7 @@ public PushConnection openPush() throws NotSupportedException, TransportExceptio
         } catch (IOException e) {
             throw new TransportException("Failed to open a fetch connection", e);
         } catch (InterruptedException e) {
+            Thread.currentThread().interrupt();
             throw new TransportException("Failed to open a fetch connection", e);
         }
 
@@ -100,15 +102,13 @@ public GitFetchTask(Pipe l2r, Pipe r2l) {
         }
 
         public Void invoke(File f, VirtualChannel channel) throws IOException, InterruptedException {
-            Repository repo = new FileRepositoryBuilder().setWorkTree(f).build();
-            try {
+            try (Repository repo = new FileRepositoryBuilder().setWorkTree(f).build()) {
                 final UploadPack rp = new UploadPack(repo);
                 rp.upload(new BufferedInputStream(l2r.getIn()), new BufferedOutputStream(r2l.getOut()), null);
                 return null;
             } finally {
                 IOUtils.closeQuietly(l2r.getIn());
                 IOUtils.closeQuietly(r2l.getOut());
-                repo.close();
             }
         }
     }
@@ -123,15 +123,13 @@ public GitPushTask(Pipe l2r, Pipe r2l) {
         }
 
         public Void invoke(File f, VirtualChannel channel) throws IOException, InterruptedException {
-            Repository repo = new FileRepositoryBuilder().setWorkTree(f).build();
-            try {
+            try (Repository repo = new FileRepositoryBuilder().setWorkTree(f).build()) {
                 final ReceivePack rp = new ReceivePack(repo);
                 rp.receive(new BufferedInputStream(l2r.getIn()), new BufferedOutputStream(r2l.getOut()), null);
                 return null;
             } finally {
                 IOUtils.closeQuietly(l2r.getIn());
                 IOUtils.closeQuietly(r2l.getOut());
-                repo.close();
             }
         }
     }

src/main/java/org/jenkinsci/plugins/gitserver/FileBackedHttpGitRepository.java

@@ -1,15 +1,16 @@
 package org.jenkinsci.plugins.gitserver;
 
+import jakarta.servlet.http.HttpServletRequest;
 import java.io.File;
 import java.io.IOException;
 import java.io.PrintWriter;
 import java.io.StringWriter;
-import java.util.Collection;
+import java.nio.file.FileAlreadyExistsException;
+import java.nio.file.Files;
+import java.nio.file.Path;
 import java.util.logging.Level;
 import java.util.logging.Logger;
-import javax.servlet.http.HttpServletRequest;
 import jenkins.model.Jenkins;
-import org.acegisecurity.Authentication;
 import org.eclipse.jgit.api.AddCommand;
 import org.eclipse.jgit.api.CommitCommand;
 import org.eclipse.jgit.api.Git;
@@ -19,12 +20,11 @@
 import org.eclipse.jgit.lib.PersonIdent;
 import org.eclipse.jgit.lib.Repository;
 import org.eclipse.jgit.storage.file.FileRepositoryBuilder;
-import org.eclipse.jgit.transport.PostReceiveHook;
-import org.eclipse.jgit.transport.ReceiveCommand;
 import org.eclipse.jgit.transport.ReceivePack;
 import org.eclipse.jgit.transport.UploadPack;
 import org.eclipse.jgit.transport.resolver.ServiceNotAuthorizedException;
 import org.eclipse.jgit.transport.resolver.ServiceNotEnabledException;
+import org.springframework.security.core.Authentication;
 
 /**
  * Convenient subtype of {@link HttpGitRepository} where the repository
@@ -38,19 +38,28 @@ public abstract class FileBackedHttpGitRepository extends HttpGitRepository {
      * Directory of the local workspace on the controller.
      * There will be "./.git" that hosts the actual repository.
      */
-    public final File workspace;
+    public final Path workspace;
 
-    protected FileBackedHttpGitRepository(File workspace) {
+    protected FileBackedHttpGitRepository(Path workspace) {
         this.workspace = workspace;
-        if (!workspace.exists() && !workspace.mkdirs()) {
-            LOGGER.log(Level.WARNING, "Cannot create a workspace in {0}", workspace);
+        try {
+            Files.createDirectory(workspace);
+        } catch (FileAlreadyExistsException ignored) {
+            // don't need to worry about this; if it already exists, we don't care!
+        } catch (IOException e) {
+            LOGGER.log(Level.WARNING, e, () -> "Cannot create a workspace in " + workspace);
         }
     }
 
+    protected FileBackedHttpGitRepository(File workspace) {
+        this(workspace.toPath());
+    }
+
     @Override
     public Repository openRepository() throws IOException {
         checkPullPermission();
-        Repository r = new FileRepositoryBuilder().setWorkTree(workspace).build();
+        Repository r =
+                new FileRepositoryBuilder().setWorkTree(workspace.toFile()).build();
 
         // if the repository doesn't exist, create it
         if (!r.getObjectDatabase().exists()) {
@@ -62,6 +71,7 @@ public Repository openRepository() throws IOException {
     /**
      * Called when there's no .git directory to create one.
      *
+     * <p>
      * This implementation also imports whatever currently in there into the repository.
      */
     protected void createInitialRepository(Repository r) throws IOException {
@@ -79,14 +89,15 @@ protected void createInitialRepository(Repository r) throws IOException {
             co.setMessage("Initial import of the existing contents");
             co.call();
         } catch (GitAPIException e) {
-            LOGGER.log(Level.WARNING, "Initial import of " + workspace + " into Git repository failed", e);
+            LOGGER.log(Level.WARNING, e, () -> "Initial import of " + workspace + " into Git repository failed");
         }
     }
 
     /**
      * This default implementation allows read access to anyone
      * who can access the HTTP URL this repository is bound to.
      *
+     * <p>
      * For example, if this object is used as a project action,
      * and the project isn't readable to Alice, then Alice won't be
      * able to pull from this repository (think of a POSIX file system
@@ -104,7 +115,7 @@ public UploadPack createUploadPack(HttpServletRequest context, Repository db)
     @Override
     public ReceivePack createReceivePack(HttpServletRequest context, Repository db)
             throws ServiceNotEnabledException, ServiceNotAuthorizedException {
-        Authentication a = Jenkins.getAuthentication();
+        Authentication a = Jenkins.getAuthentication2();
 
         ReceivePack rp = createReceivePack(db);
 
@@ -119,15 +130,13 @@ public ReceivePack createReceivePack(Repository db) {
         ReceivePack rp = new ReceivePack(db);
 
         // update userContent after the push
-        rp.setPostReceiveHook(new PostReceiveHook() {
-            public void onPostReceive(ReceivePack rp, Collection<ReceiveCommand> commands) {
-                try {
-                    updateWorkspace(rp.getRepository());
-                } catch (Exception e) {
-                    StringWriter sw = new StringWriter();
-                    e.printStackTrace(new PrintWriter(sw));
-                    rp.sendMessage("Failed to update workspace: " + sw);
-                }
+        rp.setPostReceiveHook((rp1, commands) -> {
+            try {
+                updateWorkspace(rp1.getRepository());
+            } catch (Exception e) {
+                StringWriter sw = new StringWriter();
+                e.printStackTrace(new PrintWriter(sw));
+                rp1.sendMessage("Failed to update workspace: " + sw);
             }
         });
         return rp;