Skip to content

Commit 1de5b83

Browse files
authored
[JENKINS-73315] Adapt OkHttp API for Jetty 12 (EE 9) (#205)
1 parent 02a7d30 commit 1de5b83

File tree

2 files changed

+14
-8
lines changed

2 files changed

+14
-8
lines changed

pom.xml

+9-3
Original file line numberDiff line numberDiff line change
@@ -28,17 +28,17 @@
2828
<revision>4.11.0</revision>
2929
<changelist>999999-SNAPSHOT</changelist>
3030
<gitHubRepo>jenkinsci/${project.artifactId}-plugin</gitHubRepo>
31-
<jenkins.version>2.472</jenkins.version>
31+
<jenkins.version>2.475</jenkins.version>
3232
<no-test-jar>false</no-test-jar>
3333
<!--
3434
beware https://github.com/jenkinsci/plugin-pom/issues/705 and https://github.com/jenkinsci/plugin-pom/issues/707
3535
if updating these ensure the used transitive deps that have their version managed are updated as direct dependencies and any unused ones are removed.
3636
-->
3737
<okio.version>3.5.0</okio.version>
3838
<kotlin.version>1.9.22</kotlin.version>
39-
<!--TODO Until is included in parent pom -->
39+
<!-- TODO JENKINS-73339 until in parent POM -->
40+
<jenkins-test-harness.version>2265.v3da_49c8134d6</jenkins-test-harness.version>
4041
<maven.compiler.release>17</maven.compiler.release>
41-
<jenkins-test-harness.version>2250.v03a_1295b_0a_30</jenkins-test-harness.version>
4242
</properties>
4343

4444
<dependencyManagement>
@@ -64,6 +64,12 @@
6464
<scope>import</scope>
6565
<type>pom</type>
6666
</dependency>
67+
<!-- TODO JENKINS-73339 until in parent POM, work around https://github.com/jenkinsci/plugin-pom/issues/936 -->
68+
<dependency>
69+
<groupId>jakarta.servlet</groupId>
70+
<artifactId>jakarta.servlet-api</artifactId>
71+
<version>5.0.0</version>
72+
</dependency>
6773
</dependencies>
6874
</dependencyManagement>
6975

src/test/java/jenkins/plugins/github/api/mock/MockGitHub.java

+5-5
Original file line numberDiff line numberDiff line change
@@ -22,18 +22,18 @@
2222
import java.util.concurrent.ThreadPoolExecutor;
2323
import java.util.concurrent.TimeUnit;
2424
import java.util.concurrent.atomic.AtomicLong;
25-
import javax.servlet.ServletException;
25+
import jakarta.servlet.ServletException;
2626
import org.eclipse.jetty.server.HttpConfiguration;
2727
import org.eclipse.jetty.server.HttpConnectionFactory;
2828
import org.eclipse.jetty.server.Server;
2929
import org.eclipse.jetty.server.ServerConnector;
30-
import org.eclipse.jetty.ee8.servlet.ServletContextHandler;
30+
import org.eclipse.jetty.ee9.servlet.ServletContextHandler;
3131
import org.jvnet.hudson.test.ThreadPoolImpl;
3232
import org.kohsuke.stapler.HttpResponse;
3333
import org.kohsuke.stapler.QueryParameter;
3434
import org.kohsuke.stapler.Stapler;
35-
import org.kohsuke.stapler.StaplerRequest;
36-
import org.kohsuke.stapler.StaplerResponse;
35+
import org.kohsuke.stapler.StaplerRequest2;
36+
import org.kohsuke.stapler.StaplerResponse2;
3737

3838
public class MockGitHub implements Closeable {
3939
private AtomicLong nextId = new AtomicLong();
@@ -178,7 +178,7 @@ public List<MockOwner<?>> owners() {
178178
public HttpResponse doRepositories(final @QueryParameter long since) {
179179
return new HttpResponse() {
180180
@Override
181-
public void generateResponse(StaplerRequest req, StaplerResponse rsp, Object node)
181+
public void generateResponse(StaplerRequest2 req, StaplerResponse2 rsp, Object node)
182182
throws IOException, ServletException {
183183
List<MockRepository> repositories = new ArrayList<>();
184184
for (MockOwner<?> o : owners()) {

0 commit comments

Comments
 (0)