Skip to content

Commit 10d8365

Browse files
committed
Ref #34717: replace javax.json by jakarta
1 parent 455a404 commit 10d8365

8 files changed

Lines changed: 27 additions & 27 deletions

File tree

pom.xml

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,8 @@
3838
<bootstrap.version>3.4.1</bootstrap.version>
3939
<jquery.version>3.7.1</jquery.version>
4040
<fontawesome.version>4.7.0</fontawesome.version>
41-
<javax-json.version>1.1.4</javax-json.version>
41+
<jakarta-json-api.version>2.1.3</jakarta-json-api.version>
42+
<glassfish-jakarta-json.version>2.0.1</glassfish-jakarta-json.version>
4243
<!-- Plugin versions -->
4344
<maven.license-plugin.version>4.6</maven.license-plugin.version>
4445
<maven.build-helper-maven.plugin.version>3.6.0</maven.build-helper-maven.plugin.version>
@@ -381,20 +382,19 @@
381382
<groupId>software.amazon.awssdk</groupId>
382383
<artifactId>ecs</artifactId>
383384
</dependency>
384-
385385
<dependency>
386-
<groupId>javax.json</groupId>
387-
<artifactId>javax.json-api</artifactId>
388-
<version>${javax-json.version}</version>
386+
<groupId>org.glassfish</groupId>
387+
<artifactId>jakarta.json</artifactId>
388+
<version>${glassfish-jakarta-json.version}</version>
389389
</dependency>
390390
<dependency>
391-
<groupId>org.glassfish</groupId>
392-
<artifactId>javax.json</artifactId>
393-
<version>${javax-json.version}</version>
391+
<groupId>jakarta.json</groupId>
392+
<artifactId>jakarta.json-api</artifactId>
393+
<version>${jakarta-json-api.version}</version>
394394
</dependency>
395395
<dependency>
396396
<groupId>com.fasterxml.jackson.datatype</groupId>
397-
<artifactId>jackson-datatype-jsr353</artifactId>
397+
<artifactId>jackson-datatype-jakarta-jsonp</artifactId>
398398
</dependency>
399399
<dependency>
400400
<groupId>com.fasterxml.jackson.dataformat</groupId>

src/main/java/eu/openanalytics/containerproxy/ContainerProxyApplication.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
*/
2121
package eu.openanalytics.containerproxy;
2222

23-
import com.fasterxml.jackson.datatype.jsr353.JSR353Module;
23+
import com.fasterxml.jackson.datatype.jsonp.JSONPModule;
2424
import eu.openanalytics.containerproxy.backend.dispatcher.proxysharing.ProxySharingDispatcher;
2525
import eu.openanalytics.containerproxy.model.spec.ProxySpec;
2626
import eu.openanalytics.containerproxy.service.hearbeat.ActiveProxiesService;
@@ -349,8 +349,8 @@ public FilterRegistrationBean<FormContentFilter> registration2(FormContentFilter
349349
* @return
350350
*/
351351
@Bean
352-
public JSR353Module jsr353Module() {
353-
return new JSR353Module();
352+
public JSONPModule jsonpModule() {
353+
return new JSONPModule();
354354
}
355355

356356
/**

src/main/java/eu/openanalytics/containerproxy/backend/kubernetes/KubernetesBackend.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
import com.fasterxml.jackson.core.JsonProcessingException;
2424
import com.fasterxml.jackson.databind.ObjectMapper;
2525
import com.fasterxml.jackson.dataformat.yaml.YAMLFactory;
26-
import com.fasterxml.jackson.datatype.jsr353.JSR353Module;
26+
import com.fasterxml.jackson.datatype.jsonp.JSONPModule;
2727
import com.google.common.base.Splitter;
2828
import eu.openanalytics.containerproxy.ContainerFailedToStartException;
2929
import eu.openanalytics.containerproxy.backend.AbstractContainerBackend;
@@ -90,6 +90,7 @@
9090
import io.fabric8.kubernetes.client.dsl.base.PatchType;
9191
import io.fabric8.kubernetes.client.readiness.Readiness;
9292
import io.fabric8.kubernetes.client.utils.Serialization;
93+
import jakarta.json.JsonPatch;
9394
import org.apache.commons.io.IOUtils;
9495
import org.apache.commons.lang3.StringUtils;
9596
import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
@@ -98,7 +99,6 @@
9899

99100
import javax.annotation.PostConstruct;
100101
import javax.inject.Inject;
101-
import javax.json.JsonPatch;
102102
import java.io.ByteArrayInputStream;
103103
import java.io.IOException;
104104
import java.io.InputStream;
@@ -526,7 +526,7 @@ private JsonPatch readPatchFromSpec(String patchAsString) throws JsonProcessingE
526526
}
527527

528528
ObjectMapper yamlReader = new ObjectMapper(new YAMLFactory());
529-
yamlReader.registerModule(new JSR353Module());
529+
yamlReader.registerModule(new JSONPModule());
530530
return yamlReader.readValue(patchAsString, JsonPatch.class);
531531
}
532532

src/main/java/eu/openanalytics/containerproxy/backend/kubernetes/PodPatcher.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,17 +23,17 @@
2323
import com.fasterxml.jackson.core.JsonProcessingException;
2424
import com.fasterxml.jackson.databind.ObjectMapper;
2525
import com.fasterxml.jackson.dataformat.yaml.YAMLFactory;
26-
import com.fasterxml.jackson.datatype.jsr353.JSR353Module;
26+
import com.fasterxml.jackson.datatype.jsonp.JSONPModule;
2727
import eu.openanalytics.containerproxy.model.runtime.Proxy;
2828
import eu.openanalytics.containerproxy.service.StructuredLogger;
2929
import io.fabric8.kubernetes.api.model.Pod;
30+
import jakarta.json.JsonPatch;
31+
import jakarta.json.JsonStructure;
3032
import org.springframework.core.env.Environment;
3133
import org.springframework.stereotype.Component;
3234

3335
import javax.annotation.PostConstruct;
3436
import javax.inject.Inject;
35-
import javax.json.JsonPatch;
36-
import javax.json.JsonStructure;
3737

3838
@Component
3939
public class PodPatcher {
@@ -48,8 +48,8 @@ public class PodPatcher {
4848

4949
@PostConstruct
5050
public void init() {
51-
mapper.registerModule(new JSR353Module());
52-
writer.registerModule(new JSR353Module());
51+
mapper.registerModule(new JSONPModule());
52+
writer.registerModule(new JSONPModule());
5353
loggingEnabled = environment.getProperty(DEBUG_PROPERTY, Boolean.class, false);
5454
}
5555

src/test/java/eu/openanalytics/containerproxy/test/e2e/app_recovery/TestAppRecovery.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,13 +23,13 @@
2323
import eu.openanalytics.containerproxy.test.helpers.ContainerSetup;
2424
import eu.openanalytics.containerproxy.test.helpers.ShinyProxyClient;
2525
import eu.openanalytics.containerproxy.test.helpers.ShinyProxyInstance;
26+
import jakarta.json.JsonObject;
2627
import org.junit.jupiter.api.Assertions;
2728
import org.junit.jupiter.api.Test;
2829
import org.junit.jupiter.params.ParameterizedTest;
2930
import org.junit.jupiter.params.provider.Arguments;
3031
import org.junit.jupiter.params.provider.MethodSource;
3132

32-
import javax.json.JsonObject;
3333
import java.util.HashSet;
3434
import java.util.Map;
3535
import java.util.stream.Stream;

src/test/java/eu/openanalytics/containerproxy/test/helpers/ShinyProxyClient.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,10 @@
2424
import com.fasterxml.jackson.databind.ObjectMapper;
2525
import eu.openanalytics.containerproxy.model.runtime.ProxyStatus;
2626
import eu.openanalytics.containerproxy.util.Retrying;
27+
import jakarta.json.Json;
28+
import jakarta.json.JsonObject;
29+
import jakarta.json.JsonObjectBuilder;
30+
import jakarta.json.JsonReader;
2731
import okhttp3.Call;
2832
import okhttp3.MediaType;
2933
import okhttp3.OkHttpClient;
@@ -33,10 +37,6 @@
3337
import org.slf4j.Logger;
3438
import org.slf4j.LoggerFactory;
3539

36-
import javax.json.Json;
37-
import javax.json.JsonObject;
38-
import javax.json.JsonObjectBuilder;
39-
import javax.json.JsonReader;
4040
import java.time.Duration;
4141
import java.util.HashMap;
4242
import java.util.HashSet;

src/test/java/eu/openanalytics/containerproxy/test/proxy/TestIntegrationOnKube.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,14 +48,14 @@
4848
import io.fabric8.kubernetes.api.model.Volume;
4949
import io.fabric8.kubernetes.api.model.VolumeMount;
5050
import io.fabric8.kubernetes.client.NamespacedKubernetesClient;
51+
import jakarta.json.JsonObject;
5152
import org.junit.jupiter.api.AfterAll;
5253
import org.junit.jupiter.api.Assertions;
5354
import org.junit.jupiter.api.Test;
5455
import org.springframework.security.authentication.UsernamePasswordAuthenticationToken;
5556
import org.springframework.security.core.Authentication;
5657
import org.springframework.security.core.authority.SimpleGrantedAuthority;
5758

58-
import javax.json.JsonObject;
5959
import java.io.ByteArrayInputStream;
6060
import java.util.Collections;
6161
import java.util.List;

src/test/java/eu/openanalytics/containerproxy/test/proxy/TestIntegrationProxySharing.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@
3939
import eu.openanalytics.containerproxy.test.helpers.TestProxySharingScaler;
4040
import eu.openanalytics.containerproxy.test.helpers.TestUtil;
4141
import eu.openanalytics.containerproxy.util.Retrying;
42+
import jakarta.json.JsonObject;
4243
import okhttp3.Request;
4344
import okhttp3.Response;
4445
import org.junit.jupiter.api.Assertions;
@@ -51,7 +52,6 @@
5152
import org.mandas.docker.client.exceptions.DockerCertificateException;
5253
import org.mandas.docker.client.exceptions.DockerException;
5354

54-
import javax.json.JsonObject;
5555
import java.io.IOException;
5656
import java.time.Duration;
5757
import java.time.Instant;

0 commit comments

Comments
 (0)