Oppgrader avhengigheter (#457)

navikt · Jan 22, 2024 · 8a0f95d · 8a0f95d
1 parent bda1465
commit 8a0f95d
Show file tree

Hide file tree

Showing 19 changed files with 91 additions and 68 deletions.
diff --git a/.nais/feature.yaml b/.nais/feature.yaml
@@ -4,6 +4,9 @@ ingresses:
   - https://bidrag-dokument-arkiv-feature.dev.intern.nav.no/
 kafka:
   pool: nav-dev
+cpu:
+  limits: 1000m
+  requests: 50m
 env:
   ACCEPTED_SERVICE_USER_AUDIENCE: srvbdarkivering,srvbisys
   STS_ISSUER_URL: https://security-token-service.dev.adeo.no

diff --git a/.nais/main.yaml b/.nais/main.yaml
@@ -4,6 +4,9 @@ ingresses:
   - https://bidrag-dokument-arkiv.dev.intern.nav.no/
 kafka:
   pool: nav-dev
+cpu:
+  limits: 1000m
+  requests: 50m
 env:
   ACCEPTED_SERVICE_USER_AUDIENCE: srvbdarkivering,srvbisys
   STS_ISSUER_URL: https://security-token-service.dev.adeo.no

diff --git a/.nais/nais.yaml b/.nais/nais.yaml
@@ -46,8 +46,10 @@ spec:
     path: /bidrag-dokument-arkiv/actuator/prometheus
   resources:
     requests:
+      cpu: {{ cpu.requests }}
       memory: 2500Mi
     limits:
+      cpu: {{ cpu.limits }}
       memory: 4096Mi
   ingresses:
   {{#each ingresses as |url|}}

diff --git a/.nais/prod.yaml b/.nais/prod.yaml
@@ -3,9 +3,11 @@ serviceuser: p
 ingresses:
   - https://bidrag-dokument-arkiv.nais.adeo.no/
   - https://bidrag-dokument-arkiv.intern.nav.no/
-  - https://bidrag-dokument-arkiv.prod-fss.nais.io
 kafka:
   pool: nav-prod
+cpu:
+  limits: 1000m
+  requests: 100m
 env:
   ACCEPTED_SERVICE_USER_AUDIENCE: srvbdarkivering,srvbisys
   STS_ISSUER_URL: https://security-token-service.nais.adeo.no

diff --git a/pom.xml b/pom.xml
@@ -27,17 +27,15 @@
 
         <!-- dependent versions other than from spring-boot -->
         <apache-avro-version>1.11.3</apache-avro-version>
-        <bidrag-commons.version>20231103132440_edabbf8</bidrag-commons.version>
-        <bidrag-commons-test.version>1.20230503150725_3c2c77b</bidrag-commons-test.version>
-        <bidrag-transport.version>20231113102624_1040a39</bidrag-transport.version>
+        <bidrag-felles.version>2024.01.19.131059</bidrag-felles.version>
         <commons-codec.version>1.16.0</commons-codec.version>
-        <graphq-dgs-client.version>8.2.1</graphq-dgs-client.version>
+        <graphq-dgs-client.version>8.2.2</graphq-dgs-client.version>
         <joark-hendelse.version>67a9be4476b63b7247cfacfaf821ab656bd2a952</joark-hendelse.version>
         <kafka-avro-serializer.version>7.5.3</kafka-avro-serializer.version>
         <logback-encoder.version>7.4</logback-encoder.version>
         <nimbus-jose-jwt.version>9.15.2</nimbus-jose-jwt.version>
-        <springdoc-openapi-ui.version>2.1.0</springdoc-openapi-ui.version>
-        <springframework-cloud.version>4.0.4</springframework-cloud.version>
+        <springdoc-openapi-ui.version>2.3.0</springdoc-openapi-ui.version>
+        <springframework-cloud.version>4.1.0</springframework-cloud.version>
         <squareup.version>4.12.0</squareup.version>
         <token-support.version>3.2.0</token-support.version>
 
@@ -79,6 +77,11 @@
                 <artifactId>scala-library</artifactId>
                 <version>2.13.12</version>
             </dependency>
+            <dependency>
+                <groupId>com.jayway.jsonpath</groupId>
+                <artifactId>json-path</artifactId>
+                <version>2.9.0</version>
+            </dependency>
         </dependencies>
     </dependencyManagement>
 
@@ -108,7 +111,7 @@
         <dependency>
             <groupId>org.apache.commons</groupId>
             <artifactId>commons-compress</artifactId>
-            <version>1.24.0</version>
+            <version>1.25.0</version>
         </dependency>
         <dependency>
             <groupId>org.apache.avro</groupId>
@@ -136,13 +139,13 @@
 
         <dependency>
             <groupId>no.nav.bidrag</groupId>
-            <artifactId>bidrag-commons</artifactId>
-            <version>${bidrag-commons.version}</version>
+            <artifactId>bidrag-commons-felles</artifactId>
+            <version>${bidrag-felles.version}</version>
         </dependency>
         <dependency>
             <groupId>no.nav.bidrag</groupId>
-            <artifactId>bidrag-transport</artifactId>
-            <version>${bidrag-transport.version}</version>
+            <artifactId>bidrag-transport-felles</artifactId>
+            <version>${bidrag-felles.version}</version>
         </dependency>
         <dependency>
             <groupId>no.nav.security</groupId>
@@ -163,6 +166,16 @@
         <dependency>
             <groupId>org.springframework.boot</groupId>
             <artifactId>spring-boot-starter-web</artifactId>
+            <exclusions>
+                <exclusion>
+                    <groupId>org.springframework.boot</groupId>
+                    <artifactId>spring-boot-starter-tomcat</artifactId>
+                </exclusion>
+            </exclusions>
+        </dependency>
+        <dependency>
+            <groupId>org.springframework.boot</groupId>
+            <artifactId>spring-boot-starter-jetty</artifactId>
         </dependency>
 
         <dependency>
@@ -251,8 +264,8 @@
         <!-- test -->
         <dependency>
             <groupId>no.nav.bidrag</groupId>
-            <artifactId>bidrag-commons-test</artifactId>
-            <version>${bidrag-commons-test.version}</version>
+            <artifactId>bidrag-commons-test-felles</artifactId>
+            <version>${bidrag-felles.version}</version>
             <scope>test</scope>
         </dependency>
 
@@ -452,7 +465,7 @@
                     <dependency>
                         <groupId>com.pinterest.ktlint</groupId>
                         <artifactId>ktlint-cli</artifactId>
-                        <version>1.0.1</version>
+                        <version>1.1.1</version>
                     </dependency>
                     <!-- additional 3rd party ruleset(s) can be specified here -->
                 </dependencies>

diff --git a/src/main/java/no/nav/bidrag/dokument/arkiv/BidragDokumentArkivConfig.kt b/src/main/java/no/nav/bidrag/dokument/arkiv/BidragDokumentArkivConfig.kt
@@ -38,6 +38,7 @@ import org.springframework.context.annotation.Import
 import org.springframework.context.annotation.Scope
 import org.springframework.http.HttpHeaders
 import org.springframework.http.MediaType
+import org.springframework.http.client.observation.DefaultClientRequestObservationConvention
 import org.springframework.retry.annotation.EnableRetry
 
 @Configuration
@@ -61,6 +62,9 @@ import org.springframework.retry.annotation.EnableRetry
     MdcFilter::class,
 )
 class BidragDokumentArkivConfig {
+    @Bean
+    fun clientRequestObservationConvention() = DefaultClientRequestObservationConvention()
+
     @Bean
     @Scope("prototype")
     fun baseSafConsumer(

diff --git a/src/main/java/no/nav/bidrag/dokument/arkiv/service/AvvikService.kt b/src/main/java/no/nav/bidrag/dokument/arkiv/service/AvvikService.kt
@@ -458,7 +458,7 @@ class AvvikService(
         if (!journalpost.harAvsenderMottaker()) {
             val brukerNavn = personConsumer.hentPerson(journalpost.bruker!!.id)
                 .orElseThrow { UgyldigAvvikException("Fant ikke person") }
-                .navn?.verdi
+                .navn
             dokarkivConsumer.endre(
                 LagreAvsenderNavnRequest(
                     journalpost.hentJournalpostIdLong()!!,

diff --git a/src/main/java/no/nav/bidrag/dokument/arkiv/service/DistribuerJournalpostService.kt b/src/main/java/no/nav/bidrag/dokument/arkiv/service/DistribuerJournalpostService.kt
@@ -339,12 +339,12 @@ class DistribuerJournalpostService(
             return null
         }
         return DistribuerTilAdresse(
-            adresseResponse.adresselinje1?.verdi,
-            adresseResponse.adresselinje2?.verdi,
-            adresseResponse.adresselinje3?.verdi,
+            adresseResponse.adresselinje1,
+            adresseResponse.adresselinje2,
+            adresseResponse.adresselinje3,
             adresseResponse.land.verdi,
-            adresseResponse.postnummer?.verdi,
-            adresseResponse.poststed?.verdi,
+            adresseResponse.postnummer,
+            adresseResponse.poststed,
         )
     }
 

diff --git a/src/main/java/no/nav/bidrag/dokument/arkiv/service/OppgaveService.kt b/src/main/java/no/nav/bidrag/dokument/arkiv/service/OppgaveService.kt
@@ -17,7 +17,6 @@ import no.nav.bidrag.dokument.arkiv.model.Discriminator
 import no.nav.bidrag.dokument.arkiv.model.OppgaveSokParametre
 import no.nav.bidrag.dokument.arkiv.model.ResourceByDiscriminator
 import no.nav.bidrag.dokument.arkiv.security.SaksbehandlerInfoManager
-import no.nav.bidrag.domene.ident.AktørId
 import no.nav.bidrag.domene.ident.Personident
 import no.nav.bidrag.transport.person.PersonDto
 import org.slf4j.LoggerFactory
@@ -102,9 +101,9 @@ class OppgaveService(
             .orElseGet {
                 PersonDto(
                     ident = Personident(gjelder),
-                    aktørId = AktørId(gjelder),
+                    aktørId = gjelder,
                 )
-            }.aktørId?.verdi
+            }.aktørId
     }
 
     private fun finnVurderDokumentOppgaverForJournalpost(journalpostId: Long): List<OppgaveData> {

diff --git a/src/main/java/no/nav/bidrag/dokument/arkiv/service/OpprettJournalpostService.kt b/src/main/java/no/nav/bidrag/dokument/arkiv/service/OpprettJournalpostService.kt
@@ -309,7 +309,9 @@ class OpprettJournalpostService(
         return dokumentDto.fysiskDokument ?: dokumentDto.dokument?.let {
             Base64.getDecoder().decode(it)
         } ?: dokumentDto.dokumentreferanse?.let {
-            LOGGER.info("Henter dokument bytedata for dokument med tittel ${dokumentDto.tittel} og dokumentreferanse ${dokumentDto.dokumentreferanse}")
+            LOGGER.info(
+                "Henter dokument bytedata for dokument med tittel ${dokumentDto.tittel} og dokumentreferanse ${dokumentDto.dokumentreferanse}",
+            )
             bidragDokumentConsumer.hentDokument(
                 it,
             )

diff --git a/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/KnyttTilAnnenSak.kt b/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/KnyttTilAnnenSak.kt
@@ -28,7 +28,9 @@ data class KnyttTilSakRequest(
         fagsakId = saksnummer,
         journalfoerendeEnhet = journalpost.journalforendeEnhet ?: "",
         bruker = KnyttTilBruker(journalpost.bruker?.id, journalpost.bruker?.type),
-        tema = tema ?: journalpost.tema ?: throw KnyttTilSakManglerTemaException("Kunne ikke knytte journalpost til annen sak. Journalpost mangler tema. "),
+        tema = tema ?: journalpost.tema ?: throw KnyttTilSakManglerTemaException(
+            "Kunne ikke knytte journalpost til annen sak. Journalpost mangler tema. ",
+        ),
     )
 }
 
@@ -45,7 +47,9 @@ data class KnyttTilGenerellSakRequest(
     constructor(journalpost: Journalpost, tema: String?) : this(
         journalfoerendeEnhet = journalpost.journalforendeEnhet ?: "9999",
         bruker = KnyttTilBruker(journalpost.bruker?.id, journalpost.bruker?.type),
-        tema = tema ?: journalpost.tema ?: throw KnyttTilSakManglerTemaException("Kunne ikke knytte journalpost til annen sak. Journalpost mangler tema. "),
+        tema = tema ?: journalpost.tema ?: throw KnyttTilSakManglerTemaException(
+            "Kunne ikke knytte journalpost til annen sak. Journalpost mangler tema. ",
+        ),
     )
 }
 

diff --git a/src/main/resources/application.yaml b/src/main/resources/application.yaml
@@ -95,7 +95,7 @@ no.nav.security.jwt:
   issuer.aad:
     discovery_url: https://login.microsoftonline.com/${AZURE_APP_TENANT_ID}/v2.0/.well-known/openid-configuration
     accepted_audience: ${AZURE_APP_CLIENT_ID}, api://${AZURE_APP_CLIENT_ID}
-    proxy_url: http://webproxy.nais:8088
+#    proxy_url: http://webproxy.nais:8088
 
 
 spring.kafka:

diff --git a/src/test/java/no/nav/bidrag/dokument/arkiv/controller/AbstractControllerTest.kt b/src/test/java/no/nav/bidrag/dokument/arkiv/controller/AbstractControllerTest.kt
@@ -8,7 +8,6 @@ import no.nav.bidrag.dokument.arkiv.BidragDokumentArkivConfig
 import no.nav.bidrag.dokument.arkiv.BidragDokumentArkivTest
 import no.nav.bidrag.dokument.arkiv.stubs.Stubs
 import no.nav.bidrag.dokument.arkiv.stubs.X_ENHET_HEADER
-import no.nav.bidrag.domene.ident.AktørId
 import no.nav.bidrag.domene.ident.Personident
 import no.nav.bidrag.transport.person.PersonDto
 import no.nav.security.token.support.spring.test.EnableMockOAuth2Server
@@ -36,7 +35,7 @@ import org.springframework.test.context.ActiveProfiles
 @EnableMockOAuth2Server
 abstract class AbstractControllerTest {
     protected var PERSON_IDENT = Personident("12345678910")
-    protected var AKTOR_IDENT = AktørId("92345678910")
+    protected var AKTOR_IDENT = "92345678910"
     protected var responseJournalpostJson = "journalpostSafResponse.json"
     protected var responseJournalpostJsonUtgaaende = "journalpostSafUtgaaendeResponse.json"
     protected var responseJournalpostJsonWithReturDetaljer =

diff --git a/src/test/java/no/nav/bidrag/dokument/arkiv/controller/AvvikControllerTest.kt b/src/test/java/no/nav/bidrag/dokument/arkiv/controller/AvvikControllerTest.kt
@@ -28,7 +28,6 @@ import no.nav.bidrag.dokument.arkiv.stubs.createOppgaveDataWithJournalpostId
 import no.nav.bidrag.dokument.arkiv.stubs.opprettSafResponse
 import no.nav.bidrag.dokument.arkiv.stubs.opprettUtgaendeDistribuertSafResponse
 import no.nav.bidrag.dokument.arkiv.stubs.opprettUtgaendeSafResponse
-import no.nav.bidrag.domene.streng.FulltNavn
 import no.nav.bidrag.transport.dokument.AvvikType
 import no.nav.bidrag.transport.dokument.Avvikshendelse
 import no.nav.bidrag.transport.dokument.BehandleAvvikshendelseResponse
@@ -636,7 +635,7 @@ class AvvikControllerTest : AbstractControllerTest() {
             PersonDto(
                 PERSON_IDENT,
                 aktørId = AKTOR_IDENT,
-                navn = FulltNavn("Personnavn"),
+                navn = "Personnavn",
             ),
             HttpStatus.OK,
         )

diff --git a/src/test/java/no/nav/bidrag/dokument/arkiv/controller/DistribuerControllerTest.kt b/src/test/java/no/nav/bidrag/dokument/arkiv/controller/DistribuerControllerTest.kt
@@ -27,13 +27,9 @@ import no.nav.bidrag.dokument.arkiv.stubs.JOURNALPOST_ID
 import no.nav.bidrag.dokument.arkiv.stubs.createDistribuerTilAdresse
 import no.nav.bidrag.dokument.arkiv.stubs.opprettSafResponse
 import no.nav.bidrag.dokument.arkiv.stubs.opprettUtgaendeSafResponse
-import no.nav.bidrag.domene.enums.Adressetype
-import no.nav.bidrag.domene.streng.Adresselinje1
-import no.nav.bidrag.domene.streng.Adresselinje2
-import no.nav.bidrag.domene.streng.Landkode2
-import no.nav.bidrag.domene.streng.Landkode3
-import no.nav.bidrag.domene.streng.Postnummer
-import no.nav.bidrag.domene.streng.Poststed
+import no.nav.bidrag.domene.enums.adresse.Adressetype
+import no.nav.bidrag.domene.land.Landkode2
+import no.nav.bidrag.domene.land.Landkode3
 import no.nav.bidrag.transport.dokument.DistribuerJournalpostRequest
 import no.nav.bidrag.transport.dokument.DistribuerJournalpostResponse
 import no.nav.bidrag.transport.dokument.DistribuerTilAdresse
@@ -847,11 +843,11 @@ internal class DistribuerControllerTest : AbstractControllerTest() {
         val headersMedEnhet = HttpHeaders()
         val postadresse = PersonAdresseDto(
             adressetype = Adressetype.BOSTEDSADRESSE,
-            adresselinje1 = Adresselinje1("Ramsegata 1"),
-            adresselinje2 = Adresselinje2("Bakredør"),
+            adresselinje1 = "Ramsegata 1",
+            adresselinje2 = "Bakredør",
             adresselinje3 = null,
-            postnummer = Postnummer("3939"),
-            poststed = Poststed("OSLO"),
+            postnummer = "3939",
+            poststed = "OSLO",
             land = Landkode2("NO"),
             land3 = Landkode3("NOR"),
         )
@@ -893,12 +889,12 @@ internal class DistribuerControllerTest : AbstractControllerTest() {
                             "BI01A01",
                             null,
                             DistribuerTilAdresse(
-                                postadresse.adresselinje1?.verdi,
-                                postadresse.adresselinje2?.verdi,
-                                postadresse.adresselinje3?.verdi,
+                                postadresse.adresselinje1,
+                                postadresse.adresselinje2,
+                                postadresse.adresselinje3,
                                 postadresse.land.verdi,
-                                postadresse.postnummer?.verdi,
-                                postadresse.poststed?.verdi,
+                                postadresse.postnummer,
+                                postadresse.poststed,
                             ),
                             batchId,
                         ),

diff --git a/src/test/java/no/nav/bidrag/dokument/arkiv/hendelser/JoarkHendelseTest.kt b/src/test/java/no/nav/bidrag/dokument/arkiv/hendelser/JoarkHendelseTest.kt
@@ -23,7 +23,6 @@ import no.nav.bidrag.dokument.arkiv.stubs.DOKUMENT_1_ID
 import no.nav.bidrag.dokument.arkiv.stubs.DOKUMENT_1_TITTEL
 import no.nav.bidrag.dokument.arkiv.stubs.Stubs
 import no.nav.bidrag.dokument.arkiv.stubs.opprettSafResponse
-import no.nav.bidrag.domene.ident.AktørId
 import no.nav.bidrag.domene.ident.Personident
 import no.nav.bidrag.transport.dokument.HendelseType
 import no.nav.bidrag.transport.dokument.JournalpostHendelse
@@ -361,7 +360,7 @@ class JoarkHendelseTest {
         stubs.mockSafResponseTilknyttedeJournalposter(listOf())
         stubs.mockBidragOrganisasjonSaksbehandler()
         stubs.mockPersonResponse(
-            PersonDto(Personident("123"), aktørId = AktørId("12321")),
+            PersonDto(Personident("123"), aktørId = "12321"),
             HttpStatus.OK,
         )
 
@@ -437,7 +436,7 @@ class JoarkHendelseTest {
         val journalpostId = 123213L
         stubs.mockSts()
         stubs.mockPersonResponse(
-            PersonDto(Personident("123"), aktørId = AktørId("12321")),
+            PersonDto(Personident("123"), aktørId = "12321"),
             HttpStatus.OK,
         )
         stubs.mockSafResponseTilknyttedeJournalposter(

diff --git a/src/test/java/no/nav/bidrag/dokument/arkiv/hendelser/OppgaveHendelseListenerTest.kt b/src/test/java/no/nav/bidrag/dokument/arkiv/hendelser/OppgaveHendelseListenerTest.kt
@@ -376,7 +376,9 @@ class OppgaveHendelseListenerTest {
                 journalpostId,
                 "\"tilleggsopplysninger\":" +
                     "[{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"},{\"nokkel\":\"origDistDigitalt\",\"verdi\":\"true\"}," +
-                    "{\"nokkel\":\"retur0_${DateUtils.formatDate(LocalDate.now())}\",\"verdi\":\"Distribusjon feilet, mottaker mangler postadresse\"}]",
+                    "{\"nokkel\":\"retur0_${DateUtils.formatDate(
+                        LocalDate.now(),
+                    )}\",\"verdi\":\"Distribusjon feilet, mottaker mangler postadresse\"}]",
             )
         }
     }
@@ -433,7 +435,9 @@ class OppgaveHendelseListenerTest {
                     "\"tilleggsopplysninger\":" +
                         "[{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"}," +
                         "{\"nokkel\":\"Lretur0_2020-01-02\",\"verdi\":\"En god begrunnelse for hvorfor dokument kom i retur\"}," +
-                        "{\"nokkel\":\"retur0_${DateUtils.formatDate(LocalDate.now())}\",\"verdi\":\"En annen god begrunnelse for hvorfor dokument kom i retur\"}",
+                        "{\"nokkel\":\"retur0_${DateUtils.formatDate(
+                            LocalDate.now(),
+                        )}\",\"verdi\":\"En annen god begrunnelse for hvorfor dokument kom i retur\"}",
                 )
             },
         )