6491: Bump SpringBoot 2.6.8 -> 3.2.3 (#642)

Co-authored-by: Marton Skjæveland <marton.skjeveland@nav.no>
navikt · Apr 15, 2024 · aaaad4e · aaaad4e
1 parent 6ba116a
commit aaaad4e
Show file tree

Hide file tree

Showing 38 changed files with 223 additions and 424 deletions.
diff --git a/.editorconfig b/.editorconfig
@@ -9,7 +9,7 @@ indent_size = 4
 trim_trailing_whitespace = true
 
 [{*.java, *.kt, *.kts}]
-ij_java_imports_layout = java.**,javax.**,|,*,|,$*
+ij_java_imports_layout = java.**, jakarta.**, |, *, |, $*
 ij_java_layout_static_imports_separately = true
 ij_java_class_count_to_use_import_on_demand = 5
 ij_java_names_count_to_use_import_on_demand = 3

diff --git a/melosys-eessi-app/pom.xml b/melosys-eessi-app/pom.xml
@@ -50,11 +50,6 @@
             <artifactId>spring-kafka</artifactId>
             <version>${spring-kafka.version}</version>
         </dependency>
-        <dependency>
-            <groupId>ch.qos.logback</groupId>
-            <artifactId>logback-classic</artifactId>
-            <version>${logback-classic.version}</version>
-        </dependency>
         <dependency>
             <groupId>net.logstash.logback</groupId>
             <artifactId>logstash-logback-encoder</artifactId>
@@ -74,26 +69,11 @@
             <artifactId>micrometer-jvm-extras</artifactId>
             <version>${micrometer-jvm-extras.version}</version>
         </dependency>
-        <dependency>
-            <groupId>com.nimbusds</groupId>
-            <artifactId>nimbus-jose-jwt</artifactId>
-            <version>${nimbus-jose-jwt.version}</version>
-        </dependency>
         <dependency>
             <groupId>org.projectlombok</groupId>
             <artifactId>lombok</artifactId>
             <version>${lombok.version}</version>
         </dependency>
-        <dependency>
-            <groupId>io.springfox</groupId>
-            <artifactId>springfox-swagger2</artifactId>
-            <version>${springfox-swagger2.version}</version>
-        </dependency>
-        <dependency>
-            <groupId>io.springfox</groupId>
-            <artifactId>springfox-swagger-ui</artifactId>
-            <version>${springfox-swagger2.version}</version>
-        </dependency>
         <dependency>
             <groupId>org.postgresql</groupId>
             <artifactId>postgresql</artifactId>
@@ -146,16 +126,22 @@
             <version>${shedlock-version}</version>
         </dependency>
         <dependency>
-            <groupId>com.vladmihalcea</groupId>
-            <artifactId>hibernate-types-52</artifactId>
-            <version>${hibernate-types.version}</version>
+            <groupId>io.hypersistence</groupId>
+            <artifactId>hypersistence-utils-hibernate-63</artifactId>
+            <version>${hypersistence-utils.version}</version>
         </dependency>
         <dependency>
             <groupId>io.getunleash</groupId>
             <artifactId>unleash-client-java</artifactId>
             <version>${unleash.version}</version>
         </dependency>
 
+        <dependency>
+            <groupId>org.springdoc</groupId>
+            <artifactId>springdoc-openapi-starter-webmvc-ui</artifactId>
+            <version>${springdoc-openapi.version}</version>
+        </dependency>
+
         <!-- TEST -->
         <dependency>
             <groupId>commons-io</groupId>

diff --git a/melosys-eessi-app/src/main/java/no/nav/melosys/eessi/config/ApplicationConfig.java b/melosys-eessi-app/src/main/java/no/nav/melosys/eessi/config/ApplicationConfig.java
@@ -8,7 +8,7 @@
 
 @Configuration
 @EnableConfigurationProperties(AppCredentials.class)
-@EnableJwtTokenValidation(ignore={"org.springframework", "springfox.documentation"})
+@EnableJwtTokenValidation(ignore={"org.springframework", "org.springdoc.webmvc"})
 @EnableRetry
 @EnableJpaAuditing
 public class ApplicationConfig {

diff --git a/melosys-eessi-app/src/main/java/no/nav/melosys/eessi/config/DatabaseConfig.java b/melosys-eessi-app/src/main/java/no/nav/melosys/eessi/config/DatabaseConfig.java
@@ -1,10 +1,10 @@
 package no.nav.melosys.eessi.config;
 
-import javax.persistence.EntityManagerFactory;
-import javax.sql.DataSource;
+import jakarta.persistence.EntityManagerFactory;
 
 import com.zaxxer.hikari.HikariConfig;
 import com.zaxxer.hikari.HikariDataSource;
+import javax.sql.DataSource;
 import lombok.SneakyThrows;
 import no.nav.vault.jdbc.hikaricp.HikariCPVaultUtil;
 import org.springframework.beans.factory.annotation.Qualifier;
@@ -52,8 +52,8 @@ public DataSource userDataSource() {
     @Bean
     public FlywayConfigurationCustomizer flywayConfig(@Qualifier("adminDataSource") DataSource adminDataSource) {
         return config ->
-                config.initSql(String.format("SET ROLE \"%s-admin\"", databaseName))
-                        .dataSource(adminDataSource);
+            config.initSql(String.format("SET ROLE \"%s-admin\"", databaseName))
+                .dataSource(adminDataSource);
     }
 
     @Bean

diff --git a/melosys-eessi-app/src/main/java/no/nav/melosys/eessi/config/SwaggerConfig.java b/melosys-eessi-app/src/main/java/no/nav/melosys/eessi/config/SwaggerConfig.java
diff --git a/melosys-eessi-app/src/main/java/no/nav/melosys/eessi/controller/BucController.java b/melosys-eessi-app/src/main/java/no/nav/melosys/eessi/controller/BucController.java
@@ -4,7 +4,7 @@
 import java.util.List;
 import java.util.stream.Collectors;
 
-import io.swagger.annotations.ApiOperation;
+import io.swagger.v3.oas.annotations.responses.ApiResponse;
 import lombok.extern.slf4j.Slf4j;
 import no.nav.melosys.eessi.controller.dto.*;
 import no.nav.melosys.eessi.integration.eux.rina_api.EuxConsumer;
@@ -44,7 +44,7 @@ public BucController(@Qualifier("tokenContext") EuxService euxService,
         this.euxConsumer = euxConsumer;
     }
 
-    @ApiOperation(value = "Oppretter første SED for den spesifikke buc-typen, og sender denne hvis sendAutomatisk=true. " +
+    @ApiResponse(description = "Oppretter første SED for den spesifikke buc-typen, og sender denne hvis sendAutomatisk=true. " +
         "Sender på eksisterende BUC hvis BUCen meddeler et lovvalg med utenlandsk myndighet, og BUCen er åpen.")
     @PostMapping(
         value = "/{bucType}",
@@ -68,7 +68,7 @@ public BucOgSedOpprettetDto opprettBucOgSed(
             oppdaterEksisterende);
     }
 
-    @ApiOperation(value = "Oppretter og sender en sed på en eksisterende buc")
+    @ApiResponse(description = "Oppretter og sender en sed på en eksisterende buc")
     @PostMapping("/{rinaSaksnummer}/sed/{sedType}")
     public void sendPåEksisterendeBuc(
         @RequestBody SedDataDto sedDataDto,
@@ -81,7 +81,7 @@ public BucOgSedOpprettetDto opprettBucOgSed(
         sedService.sendPåEksisterendeBuc(sedDataDto, rinaSaksnummer, sedType);
     }
 
-    @ApiOperation(value = "Henter mottakerinstitusjoner som er satt som EESSI-klare for den spesifikke buc-type")
+    @ApiResponse(description = "Henter mottakerinstitusjoner som er satt som EESSI-klare for den spesifikke buc-type")
     @GetMapping("/{bucType}/institusjoner")
     public List<InstitusjonDto> hentMottakerinstitusjoner(@PathVariable BucType bucType,
                                                           @RequestParam(required = false) Collection<String> land) {
@@ -90,20 +90,20 @@ public List<InstitusjonDto> hentMottakerinstitusjoner(@PathVariable BucType bucT
             .collect(Collectors.toList());
     }
 
-    @ApiOperation(value = "Henter sedGrunnlag for gitt sed")
+    @ApiResponse(description = "Henter sedGrunnlag for gitt sed")
     @GetMapping("/{rinaSaksnummer}/sed/{rinaDokumentId}/grunnlag")
     public SedGrunnlagDto hentSedGrunnlag(@PathVariable String rinaSaksnummer, @PathVariable String rinaDokumentId) {
         var sed = euxService.hentSed(rinaSaksnummer, rinaDokumentId);
         return SedGrunnlagMapperFactory.getMapper(SedType.valueOf(sed.getSedType())).map(sed);
     }
 
-    @ApiOperation(value = "Oppretter en asynkron jobb som forsøker å lukke en spesifikk BUC, om den har actions som tillater det")
+    @ApiResponse(description = "Oppretter en asynkron jobb som forsøker å lukke en spesifikk BUC, om den har actions som tillater det")
     @PostMapping("/{rinaSaksnummer}/lukk")
     public void lukkBuc(@PathVariable("rinaSaksnummer") String rinaSaksnummer) {
         lukkBucService.forsøkLukkBucAsync(rinaSaksnummer);
     }
 
-    @ApiOperation(value= "Henter mulige aksjoner på en Buc")
+    @ApiResponse(description = "Henter mulige aksjoner på en Buc")
     @GetMapping("/{rinaSaksnummer}/aksjoner")
     public Collection<String> hentMuligeBucHandlinger(@PathVariable("rinaSaksnummer") String rinaSaksnummer) {
         return euxConsumer.hentBucHandlinger(rinaSaksnummer);

diff --git a/...si-app/src/main/java/no/nav/melosys/eessi/controller/JournalfoeringKoblingController.java b/...si-app/src/main/java/no/nav/melosys/eessi/controller/JournalfoeringKoblingController.java
@@ -1,6 +1,6 @@
 package no.nav.melosys.eessi.controller;
 
-import io.swagger.annotations.ApiOperation;
+import io.swagger.v3.oas.annotations.responses.ApiResponse;
 import lombok.extern.slf4j.Slf4j;
 import no.nav.melosys.eessi.controller.dto.JournalpostSedKoblingDto;
 import no.nav.melosys.eessi.kafka.producers.model.MelosysEessiMelding;
@@ -24,18 +24,18 @@ public JournalfoeringKoblingController(JournalpostSedKoblingService journalpostS
         this.journalpostSedKoblingService = journalpostSedKoblingService;
     }
 
-    @ApiOperation(value = "Henter objekt som beskriver dataen mottatt i sed som journalpost er koblet til")
+    @ApiResponse(description = "Henter objekt som beskriver dataen mottatt i sed som journalpost er koblet til")
     @GetMapping("{journalpostID}/eessimelding")
     public MelosysEessiMelding hentEessiMeldingFraJournalpost(@PathVariable("journalpostID") String journalpostID) {
         return journalpostSedKoblingService.finnVedJournalpostIDOpprettMelosysEessiMelding(journalpostID)
-                .orElseThrow(() -> new NotFoundException("Finner ikke rinasak tilhørende journalpostID " + journalpostID));
+            .orElseThrow(() -> new NotFoundException("Finner ikke rinasak tilhørende journalpostID " + journalpostID));
     }
 
-    @ApiOperation(value = "Henter sed koblet til journalpost. Gir tomt svar om det ikke finnes en relasjon")
+    @ApiResponse(description = "Henter sed koblet til journalpost. Gir tomt svar om det ikke finnes en relasjon")
     @GetMapping("{journalpostID}")
     public JournalpostSedKoblingDto hentJournalpostInfo(@PathVariable("journalpostID") String journalpostID) {
         return journalpostSedKoblingService.finnVedJournalpostID(journalpostID)
-                .map(JournalpostSedKoblingDto::new)
-                .orElseGet(JournalpostSedKoblingDto::new);
+            .map(JournalpostSedKoblingDto::new)
+            .orElseGet(JournalpostSedKoblingDto::new);
     }
 }
diff --git a/melosys-eessi-app/src/main/java/no/nav/melosys/eessi/controller/KopierBucController.java b/melosys-eessi-app/src/main/java/no/nav/melosys/eessi/controller/KopierBucController.java
@@ -4,7 +4,7 @@
 import java.util.HashMap;
 import java.util.Map;
 
-import io.swagger.annotations.ApiOperation;
+import io.swagger.v3.oas.annotations.responses.ApiResponse;
 import no.nav.melosys.eessi.service.buc.KopierBucService;
 import no.nav.security.token.support.core.api.Protected;
 import org.springframework.web.bind.annotation.PostMapping;
@@ -21,7 +21,7 @@ public KopierBucController(KopierBucService kopierBucService) {
         this.kopierBucService = kopierBucService;
     }
 
-    @ApiOperation("Kopierer over første SED i en BUC til en ny BUC og lagrer saksrelasjon på samme arkivsak som tidligere BUC")
+    @ApiResponse(description = "Kopierer over første SED i en BUC til en ny BUC og lagrer saksrelasjon på samme arkivsak som tidligere BUC")
     @PostMapping("/bucer/kopier")
     public Map<String, String> kopierOverBUCer(@RequestBody Collection<String> bucer) {
         var saker = new HashMap<String, String>();

diff --git a/melosys-eessi-app/src/main/java/no/nav/melosys/eessi/controller/SakController.java b/melosys-eessi-app/src/main/java/no/nav/melosys/eessi/controller/SakController.java
@@ -4,9 +4,11 @@
 import java.util.List;
 import java.util.Optional;
 import java.util.stream.Collectors;
-import javax.websocket.server.PathParam;
 
-import io.swagger.annotations.ApiOperation;
+import io.swagger.v3.oas.annotations.responses.ApiResponse;
+
+import jakarta.websocket.server.PathParam;
+
 import lombok.extern.slf4j.Slf4j;
 import no.nav.melosys.eessi.controller.dto.BucinfoDto;
 import no.nav.melosys.eessi.controller.dto.SaksrelasjonDto;
@@ -38,43 +40,43 @@ public SakController(EuxService euxService, SaksrelasjonService saksrelasjonServ
 
 
     @GetMapping("/{gsakSaksnummer}/bucer")
-    @ApiOperation(value = "Henter bucer tilknyttet en sak")
+    @ApiResponse(description = "Henter bucer tilknyttet en sak")
     public List<BucinfoDto> hentTilknyttedeBucer(@PathVariable Long gsakSaksnummer,
                                                  @RequestParam(required = false) List<String> statuser) {
         return saksrelasjonService.finnVedGsakSaksnummer(gsakSaksnummer).stream()
-                .map(FagsakRinasakKobling::getRinaSaksnummer)
-                .map(euxService::finnBUC)
-                .flatMap(Optional::stream)
-                .map(buc -> BucinfoDto.av(buc, statuser, euxService.hentRinaUrl(buc.getId())))
-                .collect(Collectors.toList());
+            .map(FagsakRinasakKobling::getRinaSaksnummer)
+            .map(euxService::finnBUC)
+            .flatMap(Optional::stream)
+            .map(buc -> BucinfoDto.av(buc, statuser, euxService.hentRinaUrl(buc.getId())))
+            .collect(Collectors.toList());
     }
 
     @PostMapping
-    @ApiOperation("Lagrer en saksrelasjon mellom en rinasak og en gsak-sak")
+    @ApiResponse(description = "Lagrer en saksrelasjon mellom en rinasak og en gsak-sak")
     public ResponseEntity<Void> lagreSaksrelasjon(@RequestBody SaksrelasjonDto saksrelasjonDto)
-            throws ValidationException {
+        throws ValidationException {
         validerSaksrelasjonDto(saksrelasjonDto);
 
         saksrelasjonService.lagreKobling(saksrelasjonDto.getGsakSaksnummer(),
-                saksrelasjonDto.getRinaSaksnummer(), BucType.valueOf(saksrelasjonDto.getBucType()));
+            saksrelasjonDto.getRinaSaksnummer(), BucType.valueOf(saksrelasjonDto.getBucType()));
 
         return ResponseEntity.ok().build();
     }
 
     @GetMapping
-    @ApiOperation(value = "Søker etter saksrelasjon basert på enten rinaSaksnummer eller gsakSaksnummer")
+    @ApiResponse(description = "Søker etter saksrelasjon basert på enten rinaSaksnummer eller gsakSaksnummer")
     public List<SaksrelasjonDto> hentSaksrelasjon(
-            @PathParam("rinaSaksnummer") String rinaSaksnummer,
-            @PathParam("gsakSaksnummer") Long gsakSaksnummer) throws ValidationException {
+        @PathParam("rinaSaksnummer") String rinaSaksnummer,
+        @PathParam("gsakSaksnummer") Long gsakSaksnummer) throws ValidationException {
 
         if (!StringUtils.hasText(rinaSaksnummer) && gsakSaksnummer != null) {
             return saksrelasjonService.finnVedGsakSaksnummer(gsakSaksnummer).stream()
-                    .map(SaksrelasjonDto::av)
-                    .collect(Collectors.toList());
+                .map(SaksrelasjonDto::av)
+                .collect(Collectors.toList());
         } else if (StringUtils.hasText(rinaSaksnummer) && gsakSaksnummer == null) {
             return saksrelasjonService.søkEtterSaksnummerFraRinaSaksnummer(rinaSaksnummer)
-                    .map(saksnummer -> Collections.singletonList(new SaksrelasjonDto(saksnummer, rinaSaksnummer, null)))
-                    .orElse(Collections.emptyList());
+                .map(saksnummer -> Collections.singletonList(new SaksrelasjonDto(saksnummer, rinaSaksnummer, null)))
+                .orElse(Collections.emptyList());
         }
 
         throw new ValidationException("Kun en av rinaSaksnummer og gsakSaksnummer kan spørres på");
@@ -92,7 +94,7 @@ private void validerSaksrelasjonDto(SaksrelasjonDto saksrelasjonDto) throws Vali
 
         if (eksisterende.isPresent() && !eksisterende.get().getGsakSaksnummer().equals(saksrelasjonDto.getGsakSaksnummer())) {
             throw new ValidationException("Rinasak " + saksrelasjonDto.getRinaSaksnummer() +
-                    " er allerede koblet mot gsakSaksnummer " + eksisterende.get().getGsakSaksnummer());
+                " er allerede koblet mot gsakSaksnummer " + eksisterende.get().getGsakSaksnummer());
         }
     }
 }
diff --git a/...p/src/main/java/no/nav/melosys/eessi/controller/interceptor/CorrelationIdInterceptor.java b/...p/src/main/java/no/nav/melosys/eessi/controller/interceptor/CorrelationIdInterceptor.java
@@ -2,8 +2,8 @@
 
 import java.util.Objects;
 import java.util.UUID;
-import javax.servlet.http.HttpServletRequest;
-import javax.servlet.http.HttpServletResponse;
+import jakarta.servlet.http.HttpServletRequest;
+import jakarta.servlet.http.HttpServletResponse;
 
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;