Skip to content

Commit

Permalink
Updated Jetty dependency versions (#5033)
Browse files Browse the repository at this point in the history
Signed-off-by: Maxim Nesen <maxim.nesen@oracle.com>

Co-authored-by: jansupol <jan.supol@oracle.com>
  • Loading branch information
senivam and jansupol authored Apr 11, 2022
1 parent 1295a7a commit 6b59ed8
Show file tree
Hide file tree
Showing 8 changed files with 37 additions and 25 deletions.
8 changes: 4 additions & 4 deletions containers/glassfish/jersey-gf-ejb/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -112,12 +112,12 @@
<extensions>true</extensions>
<configuration>
<instructions>
<!-- Explicitly set versions for packages from GlassFish to allow future uptake of GlassFish 5.x-->
<!-- Explicitly set versions for packages from GlassFish to allow future uptake of GlassFish 7.x-->
<Import-Package>
com.sun.*;version="[4.0,7)",
com.sun.*;version="[6.0,8)",
${jakarta.annotation.osgi.version},
org.glassfish.ejb.*;version="[4.0,7)",
org.glassfish.internal.*;version="[4.0,7)",
org.glassfish.ejb.*;version="[6.0,8)",
org.glassfish.internal.*;version="[6.0,8)",
${hk2.osgi.version},
*
</Import-Package>
Expand Down
2 changes: 1 addition & 1 deletion containers/grizzly2-servlet/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@
<configuration>
<instructions>
<Import-Package>
jakarta.servlet.*;version="[5.0,6.0)",
jakarta.servlet.*;version="[5.0,7.0)",
*
</Import-Package>
</instructions>
Expand Down
2 changes: 1 addition & 1 deletion containers/jersey-servlet-core/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@
<!-- Note: When you're changing these properties change them also in bundles/jax-rs-ri/bundle/pom.xml. -->
<Import-Package>
jakarta.persistence.*;resolution:=optional,
jakarta.servlet.*;version="[5.0,6.0)",
jakarta.servlet.*;version="[5.0,7.0)",
${jakarta.annotation.osgi.version},
*
</Import-Package>
Expand Down
2 changes: 1 addition & 1 deletion containers/jersey-servlet/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@
<instructions>
<!-- Note: When you're changing these properties change them also in bundles/jax-rs-ri/bundle/pom.xml. -->
<Import-Package>
jakarta.servlet.*;version="[5.0,6.0)",
jakarta.servlet.*;version="[5.0,7.0)",
${jakarta.annotation.osgi.version},
*
</Import-Package>
Expand Down
2 changes: 1 addition & 1 deletion containers/jetty-servlet/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@
<configuration>
<instructions>
<Import-Package>
jakarta.servlet.*;version="[5.0,6.0)",
jakarta.servlet.*;version="[5.0,7.0)",
*
</Import-Package>
</instructions>
Expand Down
2 changes: 1 addition & 1 deletion examples/osgi-http-service/bundle/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@
<configuration>
<instructions>
<Export-Package>org.glassfish.jersey.examples.osgihttpservice</Export-Package>
<Import-Package>jakarta.servlet.*;version="[5.0,6.0)",*</Import-Package>
<Import-Package>jakarta.servlet.*;version="[5.0,7.0)",*</Import-Package>
<Bundle-Activator>org.glassfish.jersey.examples.osgihttpservice.Activator</Bundle-Activator>
<Implementation-Title>jersey-osgi-http-service-bundle</Implementation-Title>
<Implementation-Version>${project.version}</Implementation-Version>
Expand Down
31 changes: 16 additions & 15 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -1829,7 +1829,7 @@
<dependency>
<groupId>jakarta.json</groupId>
<artifactId>jakarta.json-api</artifactId>
<version>${jsonp.ri.version}</version>
<version>${jakarta.jsonp.version}</version>
</dependency>
<dependency>
<groupId>org.glassfish</groupId>
Expand Down Expand Up @@ -2143,12 +2143,12 @@
<slf4j.version>1.7.21</slf4j.version>
<spring4.version>4.3.20.RELEASE</spring4.version>
<spring5.version>5.1.5.RELEASE</spring5.version>
<surefire.version>3.0.0-M3</surefire.version>
<surefire.version>3.0.0-M5</surefire.version>

<!-- Jakartified, eligible for CQ -->
<weld.version>4.0.2.Final</weld.version>
<weld3.version>3.1.7.SP1</weld3.version>
<validation.impl.version>7.0.0.Final</validation.impl.version>
<validation.impl.version>7.0.1.Final</validation.impl.version>
<!-- END of Jakartified, eligible for CQ -->
<xerces.version>2.11.0</xerces.version>

Expand All @@ -2160,36 +2160,37 @@
<!-- Jakartified -->
<cdi.api.version>3.0.0</cdi.api.version>
<ejb.version>4.0.0</ejb.version>
<grizzly2.version>3.0.0</grizzly2.version>
<grizzly2.version>3.0.1</grizzly2.version>
<grizzly.npn.version>2.0.0</grizzly.npn.version>
<hk2.version>3.0.1</hk2.version>
<hk2.version>3.0.2</hk2.version>
<jsp.version>3.0.0</jsp.version>
<jstl.version>2.0.0</jstl.version>
<jta.api.version>2.0.0</jta.api.version>
<servlet5.version>5.0.0</servlet5.version>
<istack.commons.runtime.version>4.0.0</istack.commons.runtime.version>
<jakarta.activation-api.version>2.0.0</jakarta.activation-api.version>
<jakarta.activation-api.version>2.0.1</jakarta.activation-api.version>
<jakarta.activation.version>2.0.1</jakarta.activation.version>
<jakarta.el.version>4.0.0</jakarta.el.version>
<jakarta.el.impl.version>4.0.0</jakarta.el.impl.version>
<jakarta.el.impl.version>4.0.2</jakarta.el.impl.version>
<jakarta.annotation.osgi.version>jakarta.annotation.*;version="[2.0,3)"</jakarta.annotation.osgi.version>
<jakarta.annotation.version>2.0.0</jakarta.annotation.version>
<jakarta.inject.version>2.0.0</jakarta.inject.version>
<jakarta.inject.version>2.0.1</jakarta.inject.version>
<jakarta.interceptor.version>2.0.0</jakarta.interceptor.version>
<jakarta.jsonp.version>2.0.1</jakarta.jsonp.version>
<jakarta.persistence.version>3.0.0</jakarta.persistence.version>
<jakarta.validation.api.version>3.0.0</jakarta.validation.api.version>
<jakarta.validation.api.version>3.0.0</jakarta.validation.api.version> <!--Can't be updated to 3.0.1 /OSGi incompatibility with JDK 1.8 -->
<jakarta.jaxb.api.version>3.0.1</jakarta.jaxb.api.version>
<jaxb.ri.version>3.0.2</jaxb.ri.version>
<jaxrs.api.spec.version>3.0</jaxrs.api.spec.version>
<jaxrs.api.impl.version>3.0.0</jaxrs.api.impl.version>
<jetty.version>11.0.0</jetty.version>
<jetty.plugin.version>11.0.0</jetty.plugin.version>
<jetty.version>11.0.7</jetty.version>
<jetty.plugin.version>11.0.7</jetty.plugin.version>
<jetty.servlet.api.25.version>6.1.14</jetty.servlet.api.25.version>
<jsonb.api.version>2.0.0</jsonb.api.version>
<jsonp.ri.version>2.0.0</jsonp.ri.version>
<jsonp.jaxrs.version>2.0.0</jsonp.jaxrs.version>
<moxy.version>3.0.0</moxy.version>
<yasson.version>2.0.1</yasson.version>
<jsonp.ri.version>2.0.1</jsonp.ri.version>
<jsonp.jaxrs.version>2.0.1</jsonp.jaxrs.version>
<moxy.version>3.0.2</moxy.version>
<yasson.version>2.0.3</yasson.version>
<!-- END of Jakartified -->

<javax.annotation.version>1.3.2</javax.annotation.version> <!--Deprecated, used only for @generated annotation in perf tests -->
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2015, 2020 Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2015, 2022 Oracle and/or its affiliates. All rights reserved.
*
* This program and the accompanying materials are made available under the
* terms of the Eclipse Public License v. 2.0, which is available at
Expand Down Expand Up @@ -31,6 +31,7 @@
import org.glassfish.jersey.server.ResourceConfig;
import org.glassfish.jersey.server.ServerProperties;

import org.glassfish.jersey.test.jetty.JettyTestContainerFactory;
import org.junit.Test;
import static org.junit.Assert.assertNotEquals;

Expand Down Expand Up @@ -118,12 +119,19 @@ public void testSlashesWithBeginningEmpty() {

@Test
public void testSlashesWithBeginningEmptyPathParam() {
if (JettyTestContainerFactory.class.isInstance(factory)) {
return; // since Jetty 11.0.5
}
Response result = call("///test");
assertEquals("-", result.readEntity(String.class));
}

@Test
public void testSlashesWithBeginningEmptyPathParamWithQueryParams() {
if (JettyTestContainerFactory.class.isInstance(factory)) {
return; // since Jetty 11.0.5
}

URI hostPort = UriBuilder.fromUri("http://localhost/").port(getPort()).build();
WebTarget target = client().target(hostPort).path("///testParams")
.queryParam("bar", "Container")
Expand All @@ -135,6 +143,9 @@ public void testSlashesWithBeginningEmptyPathParamWithQueryParams() {

@Test
public void testEncodedQueryParams() {
if (JettyTestContainerFactory.class.isInstance(factory)) {
return; // Jetty 11.0.5
}
URI hostPort = UriBuilder.fromUri("http://localhost/").port(getPort()).build();
WebTarget target = client().target(hostPort).path("///encoded")
.queryParam("query", "%dummy23+a");
Expand Down

0 comments on commit 6b59ed8

Please sign in to comment.