Skip to content

Commit 750b1ca

Browse files
basilwouter-admiraal-sonarsource
authored andcommitted
Migrate from EE 8 to EE 9
1 parent 671be87 commit 750b1ca

File tree

7 files changed

+23
-24
lines changed

7 files changed

+23
-24
lines changed

pom.xml

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<groupId>org.jenkins-ci.plugins</groupId>
77
<artifactId>plugin</artifactId>
88
<!-- See https://github.com/jenkinsci/plugin-pom/releases -->
9-
<version>4.88</version>
9+
<version>5.9</version>
1010
<relativePath />
1111
</parent>
1212

@@ -49,7 +49,9 @@
4949
<sonar.exclusions>target/generated-sources/**/*</sonar.exclusions>
5050
<gitRepositoryName>sonar-scanner-jenkins</gitRepositoryName>
5151
<version.artifactory.plugin>3.6.1</version.artifactory.plugin>
52-
<jenkins.version>2.462.3</jenkins.version>
52+
<!-- https://www.jenkins.io/doc/developer/plugin-development/choosing-jenkins-baseline/ -->
53+
<jenkins.baseline>2.479</jenkins.baseline>
54+
<jenkins.version>${jenkins.baseline}.3</jenkins.version>
5355
<version.cyclonedx.plugin>2.7.9</version.cyclonedx.plugin>
5456
<!-- Disable parallel tests on Travis -->
5557
<concurrency>1</concurrency>
@@ -72,8 +74,8 @@
7274
<dependencies>
7375
<dependency>
7476
<groupId>io.jenkins.tools.bom</groupId>
75-
<artifactId>bom-2.462.x</artifactId>
76-
<version>3387.v0f2773fa_3200</version>
77+
<artifactId>bom-${jenkins.baseline}.x</artifactId>
78+
<version>4023.va_eeb_b_4e45f07</version>
7779
<scope>import</scope>
7880
<type>pom</type>
7981
</dependency>
@@ -145,7 +147,6 @@
145147
<!-- needed for SonarPublisher -->
146148
<groupId>org.jenkins-ci.main</groupId>
147149
<artifactId>maven-plugin</artifactId>
148-
<version>3.23</version>
149150
<optional>true</optional>
150151
</dependency>
151152
<dependency>
@@ -169,12 +170,10 @@
169170
<groupId>org.jenkins-ci.plugins</groupId>
170171
<artifactId>script-security</artifactId>
171172
<optional>true</optional>
172-
<version>1368.vb_b_402e3547e7</version>
173173
</dependency>
174174
<dependency>
175175
<groupId>org.jenkins-ci.plugins</groupId>
176176
<artifactId>credentials</artifactId>
177-
<version>1381.v2c3a_12074da_b_</version>
178177
</dependency>
179178
<dependency>
180179
<groupId>org.jenkins-ci.plugins</groupId>

src/main/java/hudson/plugins/sonar/MsBuildSQRunnerInstallation.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
import jenkins.security.MasterToSlaveCallable;
4242
import net.sf.json.JSONObject;
4343
import org.kohsuke.stapler.DataBoundConstructor;
44-
import org.kohsuke.stapler.StaplerRequest;
44+
import org.kohsuke.stapler.StaplerRequest2;
4545

4646
public class MsBuildSQRunnerInstallation extends ToolInstallation implements EnvironmentSpecific<MsBuildSQRunnerInstallation>, NodeSpecific<MsBuildSQRunnerInstallation> {
4747
private static final long serialVersionUID = 1L;
@@ -110,7 +110,7 @@ public void setInstallations(MsBuildSQRunnerInstallation... installations) {
110110
}
111111

112112
@Override
113-
public MsBuildSQRunnerInstallation newInstance(StaplerRequest req, JSONObject formData) {
113+
public MsBuildSQRunnerInstallation newInstance(StaplerRequest2 req, JSONObject formData) {
114114
return (MsBuildSQRunnerInstallation) req.bindJSON(clazz, formData);
115115
}
116116

src/main/java/hudson/plugins/sonar/SonarGlobalConfiguration.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@
4545
import org.apache.commons.lang.StringUtils;
4646
import org.jenkinsci.plugins.plaincredentials.StringCredentials;
4747
import org.kohsuke.stapler.QueryParameter;
48-
import org.kohsuke.stapler.StaplerRequest;
48+
import org.kohsuke.stapler.StaplerRequest2;
4949

5050
/**
5151
* Since 2.4
@@ -140,7 +140,7 @@ public void migrateCredentials() {
140140
}
141141

142142
@Override
143-
public boolean configure(StaplerRequest req, JSONObject json) {
143+
public boolean configure(StaplerRequest2 req, JSONObject json) {
144144
List<SonarInstallation> list = req.bindJSONToList(SonarInstallation.class, json.get("inst"));
145145
boolean enableBuildWrapper = json.getBoolean("enableBuildWrapper");
146146
setInstallations(list.toArray(new SonarInstallation[list.size()]));

src/main/java/hudson/plugins/sonar/SonarRunnerInstallation.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@
4242
import jenkins.security.MasterToSlaveCallable;
4343
import net.sf.json.JSONObject;
4444
import org.kohsuke.stapler.DataBoundConstructor;
45-
import org.kohsuke.stapler.StaplerRequest;
45+
import org.kohsuke.stapler.StaplerRequest2;
4646

4747
/**
4848
* Represents a SonarQube Scanner installation in a system.
@@ -126,7 +126,7 @@ public SonarRunnerInstallation[] getInstallations() {
126126
}
127127

128128
@Override
129-
public SonarRunnerInstallation newInstance(StaplerRequest req, JSONObject formData) {
129+
public SonarRunnerInstallation newInstance(StaplerRequest2 req, JSONObject formData) {
130130
return (SonarRunnerInstallation) req.bindJSON(clazz, formData);
131131
}
132132

src/main/java/org/sonarsource/scanner/jenkins/pipeline/SonarQubeWebHook.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,13 +34,13 @@
3434
import java.util.logging.Level;
3535
import java.util.logging.Logger;
3636
import edu.umd.cs.findbugs.annotations.Nullable;
37-
import javax.servlet.http.HttpServletResponse;
37+
import jakarta.servlet.http.HttpServletResponse;
3838
import jenkins.model.Jenkins;
3939
import net.sf.json.JSONException;
4040
import net.sf.json.JSONObject;
4141
import org.apache.commons.io.IOUtils;
42-
import org.kohsuke.stapler.StaplerRequest;
43-
import org.kohsuke.stapler.StaplerResponse;
42+
import org.kohsuke.stapler.StaplerRequest2;
43+
import org.kohsuke.stapler.StaplerResponse2;
4444
import org.kohsuke.stapler.interceptor.RequirePOST;
4545

4646
@Extension
@@ -68,7 +68,7 @@ public String getUrlName() {
6868
}
6969

7070
@RequirePOST
71-
public void doIndex(StaplerRequest req, StaplerResponse rsp) throws IOException {
71+
public void doIndex(StaplerRequest2 req, StaplerResponse2 rsp) throws IOException {
7272
String payload = IOUtils.toString(req.getReader());
7373

7474
LOGGER.info("Received POST from " + req.getRemoteHost());

src/main/java/org/sonarsource/scanner/jenkins/pipeline/SonarQubeWebHookCrumbExclusion.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,10 @@
2222
import hudson.Extension;
2323
import hudson.security.csrf.CrumbExclusion;
2424
import java.io.IOException;
25-
import javax.servlet.FilterChain;
26-
import javax.servlet.ServletException;
27-
import javax.servlet.http.HttpServletRequest;
28-
import javax.servlet.http.HttpServletResponse;
25+
import jakarta.servlet.FilterChain;
26+
import jakarta.servlet.ServletException;
27+
import jakarta.servlet.http.HttpServletRequest;
28+
import jakarta.servlet.http.HttpServletResponse;
2929

3030
import static org.apache.commons.lang3.StringUtils.isEmpty;
3131

src/test/java/org/sonarsource/scanner/jenkins/pipeline/SonarQubeWebHookCrumbExclusionTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,9 @@
1919
*/
2020
package org.sonarsource.scanner.jenkins.pipeline;
2121

22-
import javax.servlet.FilterChain;
23-
import javax.servlet.http.HttpServletRequest;
24-
import javax.servlet.http.HttpServletResponse;
22+
import jakarta.servlet.FilterChain;
23+
import jakarta.servlet.http.HttpServletRequest;
24+
import jakarta.servlet.http.HttpServletResponse;
2525
import org.junit.Before;
2626
import org.junit.Test;
2727

0 commit comments

Comments
 (0)