Skip to content

Commit 088b5f2

Browse files
janfaracikalecharp
andauthored
Update BOM, update to Jakarta (#823)
* Init * Update pom.xml --------- Co-authored-by: Adrien Lecharpentier <[email protected]>
1 parent 3d7eb3e commit 088b5f2

File tree

7 files changed

+34
-26
lines changed

7 files changed

+34
-26
lines changed

pom.xml

+4-4
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
<parent>
55
<groupId>org.jenkins-ci.plugins</groupId>
66
<artifactId>plugin</artifactId>
7-
<version>4.88</version>
7+
<version>5.5</version>
88
<relativePath />
99
</parent>
1010

@@ -34,8 +34,8 @@
3434
<gitHubRepo>jenkinsci/${project.artifactId}-plugin</gitHubRepo>
3535
<hpi.compatibleSinceVersion>2.2.0</hpi.compatibleSinceVersion>
3636
<!-- https://www.jenkins.io/doc/developer/plugin-development/choosing-jenkins-baseline/ -->
37-
<jenkins.baseline>2.440</jenkins.baseline>
38-
<jenkins.version>${jenkins.baseline}.3</jenkins.version>
37+
<jenkins.baseline>2.479</jenkins.baseline>
38+
<jenkins.version>${jenkins.baseline}.1</jenkins.version>
3939
<useBeta>true</useBeta>
4040
<spotless.check.skip>false</spotless.check.skip>
4141
</properties>
@@ -45,7 +45,7 @@
4545
<dependency>
4646
<groupId>io.jenkins.tools.bom</groupId>
4747
<artifactId>bom-${jenkins.baseline}.x</artifactId>
48-
<version>3234.v5ca_5154341ef</version>
48+
<version>3893.v213a_42768d35</version>
4949
<type>pom</type>
5050
<scope>import</scope>
5151
</dependency>

src/main/java/org/jenkinsci/plugins/github_branch_source/FillErrorResponse.java

+4-6
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,12 @@
11
package org.jenkinsci.plugins.github_branch_source;
22

33
import hudson.Util;
4+
import jakarta.servlet.http.HttpServletResponse;
45
import java.io.IOException;
5-
import javax.servlet.ServletException;
6-
import javax.servlet.http.HttpServletResponse;
76
import jenkins.model.Jenkins;
87
import org.kohsuke.stapler.HttpResponse;
9-
import org.kohsuke.stapler.StaplerRequest;
10-
import org.kohsuke.stapler.StaplerResponse;
8+
import org.kohsuke.stapler.StaplerRequest2;
9+
import org.kohsuke.stapler.StaplerResponse2;
1110

1211
// TODO replace with corresponding core functionality once Jenkins core has JENKINS-42443
1312
class FillErrorResponse extends IOException implements HttpResponse {
@@ -20,8 +19,7 @@ public FillErrorResponse(String message, boolean clearList) {
2019
}
2120

2221
@Override
23-
public void generateResponse(StaplerRequest req, StaplerResponse rsp, Object node)
24-
throws IOException, ServletException {
22+
public void generateResponse(StaplerRequest2 req, StaplerResponse2 rsp, Object node) throws IOException {
2523
rsp.setStatus(HttpServletResponse.SC_INTERNAL_SERVER_ERROR);
2624
rsp.setContentType("text/html;charset=UTF-8");
2725
rsp.setHeader("X-Jenkins-Select-Error", clearList ? "clear" : "retain");

src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubConfiguration.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@
4242
import jenkins.model.Jenkins;
4343
import net.sf.json.JSONObject;
4444
import org.apache.commons.lang.StringUtils;
45-
import org.kohsuke.stapler.StaplerRequest;
45+
import org.kohsuke.stapler.StaplerRequest2;
4646

4747
@Extension
4848
public class GitHubConfiguration extends GlobalConfiguration {
@@ -60,7 +60,7 @@ public GitHubConfiguration() {
6060
}
6161

6262
@Override
63-
public boolean configure(StaplerRequest req, JSONObject json) throws FormException {
63+
public boolean configure(StaplerRequest2 req, JSONObject json) throws FormException {
6464
req.bindJSON(this, json);
6565
return true;
6666
}

src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubSCMSource.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,7 @@
5858
import hudson.util.FormValidation;
5959
import hudson.util.ListBoxModel;
6060
import hudson.util.LogTaskListener;
61+
import jakarta.servlet.http.HttpServletResponse;
6162
import java.io.Closeable;
6263
import java.io.FileNotFoundException;
6364
import java.io.IOException;
@@ -85,7 +86,6 @@
8586
import java.util.logging.Logger;
8687
import java.util.regex.Matcher;
8788
import java.util.regex.Pattern;
88-
import javax.servlet.http.HttpServletResponse;
8989
import jenkins.model.Jenkins;
9090
import jenkins.plugins.git.AbstractGitSCMSource;
9191
import jenkins.plugins.git.GitTagSCMRevision;

src/test/java/org/jenkinsci/plugins/github_branch_source/EndpointTest.java

+10-10
Original file line numberDiff line numberDiff line change
@@ -10,13 +10,13 @@
1010
import hudson.Util;
1111
import hudson.model.UnprotectedRootAction;
1212
import hudson.security.csrf.CrumbExclusion;
13+
import jakarta.servlet.FilterChain;
14+
import jakarta.servlet.ServletException;
15+
import jakarta.servlet.http.HttpServletRequest;
16+
import jakarta.servlet.http.HttpServletResponse;
1317
import java.io.IOException;
1418
import java.net.URL;
1519
import java.util.Arrays;
16-
import javax.servlet.FilterChain;
17-
import javax.servlet.ServletException;
18-
import javax.servlet.http.HttpServletRequest;
19-
import javax.servlet.http.HttpServletResponse;
2020
import jenkins.model.Jenkins;
2121
import org.htmlunit.FailingHttpStatusCodeException;
2222
import org.htmlunit.HttpMethod;
@@ -31,8 +31,8 @@
3131
import org.jvnet.hudson.test.JenkinsRule;
3232
import org.jvnet.hudson.test.MockAuthorizationStrategy;
3333
import org.jvnet.hudson.test.TestExtension;
34-
import org.kohsuke.stapler.StaplerRequest;
35-
import org.kohsuke.stapler.StaplerResponse;
34+
import org.kohsuke.stapler.StaplerRequest2;
35+
import org.kohsuke.stapler.StaplerResponse2;
3636
import org.xml.sax.SAXException;
3737

3838
public class EndpointTest {
@@ -103,7 +103,7 @@ private String appendCrumb(String url) {
103103
}
104104

105105
private String getCrumb() {
106-
return Functions.getCrumbRequestField() + "=" + Functions.getCrumb(null);
106+
return Functions.getCrumbRequestField() + "=" + Functions.getCrumb((StaplerRequest2) null);
107107
}
108108

109109
private Page post(String relative, String userName) throws Exception {
@@ -116,8 +116,8 @@ private Page post(String relative, String userName) throws Exception {
116116

117117
final WebRequest request = new WebRequest(new URL(client.getContextPath() + relative), HttpMethod.POST);
118118
request.setAdditionalHeader("Accept", client.getBrowserVersion().getHtmlAcceptHeader());
119-
request.setRequestParameters(
120-
Arrays.asList(new NameValuePair(Functions.getCrumbRequestField(), Functions.getCrumb(null))));
119+
request.setRequestParameters(Arrays.asList(
120+
new NameValuePair(Functions.getCrumbRequestField(), Functions.getCrumb((StaplerRequest2) null))));
121121
return client.getPage(request);
122122
}
123123

@@ -141,7 +141,7 @@ public String getUrlName() {
141141
return "testroot";
142142
}
143143

144-
public void doIndex(StaplerRequest request, StaplerResponse response) throws IOException {
144+
public void doIndex(StaplerRequest2 request, StaplerResponse2 response) throws IOException {
145145
visited = true;
146146
response.getWriter().println("OK");
147147
}

src/test/java/org/jenkinsci/plugins/github_branch_source/GitHubSCMBuilderTest.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
import com.cloudbees.plugins.credentials.SystemCredentialsProvider;
2121
import com.cloudbees.plugins.credentials.domains.Domain;
2222
import com.cloudbees.plugins.credentials.impl.UsernamePasswordCredentialsImpl;
23+
import hudson.model.Descriptor;
2324
import hudson.plugins.git.GitSCM;
2425
import hudson.plugins.git.Revision;
2526
import hudson.plugins.git.UserRemoteConfig;
@@ -82,7 +83,7 @@ public void createGitHubSCMSourceForTest(boolean configuredByUrl, String repoUrl
8283
}
8384

8485
@Before
85-
public void setUp() throws IOException {
86+
public void setUp() throws IOException, Descriptor.FormException {
8687
owner = j.createProject(WorkflowMultiBranchProject.class);
8788
Credentials userPasswordCredential = new UsernamePasswordCredentialsImpl(
8889
CredentialsScope.GLOBAL, "user-pass", null, "git-user", "git-secret");

src/test/java/org/jenkinsci/plugins/github_branch_source/GitHubSCMNavigatorTest.java

+11-2
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@
3737
import com.cloudbees.plugins.credentials.impl.UsernamePasswordCredentialsImpl;
3838
import edu.umd.cs.findbugs.annotations.NonNull;
3939
import edu.umd.cs.findbugs.annotations.Nullable;
40+
import hudson.model.Descriptor;
4041
import hudson.model.Item;
4142
import hudson.model.TaskListener;
4243
import hudson.model.User;
@@ -75,8 +76,16 @@ public class GitHubSCMNavigatorTest extends AbstractGitHubWireMockTest {
7576
@Mock
7677
private SCMSourceOwner scmSourceOwner;
7778

78-
private BaseStandardCredentials credentials = new UsernamePasswordCredentialsImpl(
79-
CredentialsScope.GLOBAL, "authenticated-user", null, "git-user", "git-secret");
79+
private BaseStandardCredentials credentials;
80+
81+
{
82+
try {
83+
credentials = new UsernamePasswordCredentialsImpl(
84+
CredentialsScope.GLOBAL, "authenticated-user", null, "git-user", "git-secret");
85+
} catch (Descriptor.FormException e) {
86+
throw new RuntimeException(e);
87+
}
88+
}
8089

8190
private GitHubSCMNavigator navigator;
8291

0 commit comments

Comments
 (0)