From a5e2bbb976cfd0fe4a70d92ebf56bbfbc3b3d8a4 Mon Sep 17 00:00:00 2001 From: Jens-Otto Larsen <46576810+jolarsen@users.noreply.github.com> Date: Sun, 9 Oct 2022 17:47:24 +0200 Subject: [PATCH] =?UTF-8?q?Gj=C3=B8r=20dokarkiv-klient=20synlig,=20fjern?= =?UTF-8?q?=20pdl-klient=20(#1197)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../vedtak/sikkerhet/pdp/PdpKlientImpl.java | 9 +-- ...a => AbstractSkjermetPersonGCPKlient.java} | 8 +-- .../AbstractSkjermetPersonOnPremKlient.java | 45 +++++++++++++++ .../dokarkiv/AbstractDokArkivKlient.java | 4 +- .../dokarkiv/dto/DokumentInfoOpprett.java | 5 -- .../dokarkiv/dto/Dokumentvariant.java | 56 ++++++------------- integrasjon/pdl-klient/pom.xml | 26 --------- .../felles/integrasjon/pdl/PdlKlient.java | 24 -------- .../pdl/RestSupportClientProducer.java | 22 -------- .../src/main/resources/META-INF/beans.xml | 6 -- integrasjon/pom.xml | 6 -- 11 files changed, 71 insertions(+), 140 deletions(-) rename integrasjon/arbeidsfordeling-klient/src/main/java/no/nav/vedtak/felles/integrasjon/skjerming/{AbstractSkjermetPersonKlient.java => AbstractSkjermetPersonGCPKlient.java} (89%) create mode 100644 integrasjon/arbeidsfordeling-klient/src/main/java/no/nav/vedtak/felles/integrasjon/skjerming/AbstractSkjermetPersonOnPremKlient.java delete mode 100644 integrasjon/pdl-klient/pom.xml delete mode 100644 integrasjon/pdl-klient/src/main/java/no/nav/vedtak/felles/integrasjon/pdl/PdlKlient.java delete mode 100644 integrasjon/pdl-klient/src/main/java/no/nav/vedtak/felles/integrasjon/pdl/RestSupportClientProducer.java delete mode 100644 integrasjon/pdl-klient/src/main/resources/META-INF/beans.xml diff --git a/felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/PdpKlientImpl.java b/felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/PdpKlientImpl.java index 350e7b38e..3cc5a4d89 100644 --- a/felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/PdpKlientImpl.java +++ b/felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/PdpKlientImpl.java @@ -2,7 +2,7 @@ import java.util.List; -import javax.enterprise.context.ApplicationScoped; +import javax.enterprise.context.Dependent; import javax.inject.Inject; import org.slf4j.Logger; @@ -20,15 +20,12 @@ import no.nav.vedtak.sikkerhet.pdp.xacml.XacmlResponse; import no.nav.vedtak.sikkerhet.pdp.xacml.XacmlResponseMapper; -@ApplicationScoped +@Dependent public class PdpKlientImpl implements PdpKlient { private static final Logger LOG = LoggerFactory.getLogger(PdpKlientImpl.class); - private PdpConsumer pdp; - - public PdpKlientImpl() { - } + private final PdpConsumer pdp; @Inject public PdpKlientImpl(PdpConsumer pdp) { diff --git a/integrasjon/arbeidsfordeling-klient/src/main/java/no/nav/vedtak/felles/integrasjon/skjerming/AbstractSkjermetPersonKlient.java b/integrasjon/arbeidsfordeling-klient/src/main/java/no/nav/vedtak/felles/integrasjon/skjerming/AbstractSkjermetPersonGCPKlient.java similarity index 89% rename from integrasjon/arbeidsfordeling-klient/src/main/java/no/nav/vedtak/felles/integrasjon/skjerming/AbstractSkjermetPersonKlient.java rename to integrasjon/arbeidsfordeling-klient/src/main/java/no/nav/vedtak/felles/integrasjon/skjerming/AbstractSkjermetPersonGCPKlient.java index 65e2ab59f..5f905f5f0 100644 --- a/integrasjon/arbeidsfordeling-klient/src/main/java/no/nav/vedtak/felles/integrasjon/skjerming/AbstractSkjermetPersonKlient.java +++ b/integrasjon/arbeidsfordeling-klient/src/main/java/no/nav/vedtak/felles/integrasjon/skjerming/AbstractSkjermetPersonGCPKlient.java @@ -13,19 +13,19 @@ // Extend og annoter med endpoint+default og scopes/default + tokenConfig = AzureAD_CC //@RestClientConfig(tokenConfig = TokenFlow.AZUREAD_CC, endpointProperty = "skjermet.person.rs.url", endpointDefault = "https://skjermede-personer-pip.intern.nav.no/skjermet", // scopesProperty = "skjermet.person.rs.azure.scope", scopesDefault = "api://prod-gcp.nom.skjermede-personer-pip/.default") -public abstract class AbstractSkjermetPersonKlient implements Skjerming { +public abstract class AbstractSkjermetPersonGCPKlient implements Skjerming { - private static final Logger LOG = LoggerFactory.getLogger(AbstractSkjermetPersonKlient.class); + private static final Logger LOG = LoggerFactory.getLogger(AbstractSkjermetPersonGCPKlient.class); private static final boolean TESTENV = Environment.current().isLocal(); private final RestClient client; private final RestConfig restConfig; - protected AbstractSkjermetPersonKlient() { + protected AbstractSkjermetPersonGCPKlient() { this(RestClient.client()); } - protected AbstractSkjermetPersonKlient(RestClient restClient) { + protected AbstractSkjermetPersonGCPKlient(RestClient restClient) { this.client = restClient; this.restConfig = RestConfig.forClient(this.getClass()); if (!restConfig.tokenConfig().isAzureAD()) { diff --git a/integrasjon/arbeidsfordeling-klient/src/main/java/no/nav/vedtak/felles/integrasjon/skjerming/AbstractSkjermetPersonOnPremKlient.java b/integrasjon/arbeidsfordeling-klient/src/main/java/no/nav/vedtak/felles/integrasjon/skjerming/AbstractSkjermetPersonOnPremKlient.java new file mode 100644 index 000000000..91ead8758 --- /dev/null +++ b/integrasjon/arbeidsfordeling-klient/src/main/java/no/nav/vedtak/felles/integrasjon/skjerming/AbstractSkjermetPersonOnPremKlient.java @@ -0,0 +1,45 @@ +package no.nav.vedtak.felles.integrasjon.skjerming; + +import no.nav.foreldrepenger.konfig.Environment; +import no.nav.vedtak.felles.integrasjon.rest.RestClient; +import no.nav.vedtak.felles.integrasjon.rest.RestConfig; +import no.nav.vedtak.felles.integrasjon.rest.RestRequest; + +// OBS på propertynavn vs GCP-versjon skjermet.person.onprem.rs.url=http://skjermede-personer-pip.nom/skjermet +// Extend og annoter med endpoint+default + tokenConfig = STS_CC +//@RestClientConfig(tokenConfig = TokenFlow.STS_CC, endpointProperty = "skjermet.person.onprem.rs.url", endpointDefault = "http://skjermede-personer-pip.nom/skjermet" +public abstract class AbstractSkjermetPersonOnPremKlient implements Skjerming { + + private static final boolean TESTENV = Environment.current().isLocal(); + + private final RestClient client; + private final RestConfig restConfig; + + protected AbstractSkjermetPersonOnPremKlient() { + this(RestClient.client()); + } + + protected AbstractSkjermetPersonOnPremKlient(RestClient restClient) { + this.client = restClient; + this.restConfig = RestConfig.forClient(this.getClass()); + } + + + @Override + public boolean erSkjermet(String fnr) { + if (TESTENV || fnr == null) return false; + + var request = RestRequest.newPOSTJson(new SkjermetRequestDto(fnr), restConfig.endpoint(), restConfig); + + var skjermet = client.send(request, String.class); + return "true".equalsIgnoreCase(skjermet); + } + + private boolean kallMedSjekk(RestRequest request) { + var skjermet = client.send(request, String.class); + return "true".equalsIgnoreCase(skjermet); + } + + private record SkjermetRequestDto(String personident) {} + +} diff --git a/integrasjon/dokarkiv-klient/src/main/java/no/nav/vedtak/felles/integrasjon/dokarkiv/AbstractDokArkivKlient.java b/integrasjon/dokarkiv-klient/src/main/java/no/nav/vedtak/felles/integrasjon/dokarkiv/AbstractDokArkivKlient.java index 3e1812a1c..1c7de423e 100644 --- a/integrasjon/dokarkiv-klient/src/main/java/no/nav/vedtak/felles/integrasjon/dokarkiv/AbstractDokArkivKlient.java +++ b/integrasjon/dokarkiv-klient/src/main/java/no/nav/vedtak/felles/integrasjon/dokarkiv/AbstractDokArkivKlient.java @@ -16,7 +16,7 @@ import no.nav.vedtak.felles.integrasjon.rest.RestRequest; // @RestClientConfig(tokenConfig = TokenFlow.STS_CC, endpointProperty = "dokarkiv.base.url", endpointDefault = "http://dokarkiv.default/rest/journalpostapi/v1/journalpost") -class AbstractDokArkivKlient implements DokArkiv { +public class AbstractDokArkivKlient implements DokArkiv { private static final Logger LOG = LoggerFactory.getLogger(AbstractDokArkivKlient.class); @@ -77,7 +77,7 @@ public boolean ferdigstillJournalpost(String journalpostId, String enhet) { LOG.info("DOKARKIV Ferdigstilt journalpost OK"); return true; } catch (Exception e) { - LOG.info("DOKARKIV DOKARKIV FERDIGSTILL {} feilet for {}", journalpostId, enhet, e); + LOG.info("DOKARKIV FERDIGSTILL {} feilet for {}", journalpostId, enhet, e); return false; } } diff --git a/integrasjon/dokarkiv-klient/src/main/java/no/nav/vedtak/felles/integrasjon/dokarkiv/dto/DokumentInfoOpprett.java b/integrasjon/dokarkiv-klient/src/main/java/no/nav/vedtak/felles/integrasjon/dokarkiv/dto/DokumentInfoOpprett.java index 46aee5c69..affa8cf14 100644 --- a/integrasjon/dokarkiv-klient/src/main/java/no/nav/vedtak/felles/integrasjon/dokarkiv/dto/DokumentInfoOpprett.java +++ b/integrasjon/dokarkiv-klient/src/main/java/no/nav/vedtak/felles/integrasjon/dokarkiv/dto/DokumentInfoOpprett.java @@ -43,11 +43,6 @@ public Builder leggTilDokumentvariant(Dokumentvariant dokumentvarianter) { return this; } - public Builder leggTilDokumentvariant(Dokumentvariant.Builder dokumentvariant) { - this.dokumentvarianter.add(dokumentvariant.build()); - return this; - } - public DokumentInfoOpprett build() { if (this.dokumentvarianter == null || this.dokumentvarianter.isEmpty()) { throw new IllegalArgumentException("Krever minst 1 dokumentvariant"); diff --git a/integrasjon/dokarkiv-klient/src/main/java/no/nav/vedtak/felles/integrasjon/dokarkiv/dto/Dokumentvariant.java b/integrasjon/dokarkiv-klient/src/main/java/no/nav/vedtak/felles/integrasjon/dokarkiv/dto/Dokumentvariant.java index 8304880c5..e98829fb7 100644 --- a/integrasjon/dokarkiv-klient/src/main/java/no/nav/vedtak/felles/integrasjon/dokarkiv/dto/Dokumentvariant.java +++ b/integrasjon/dokarkiv-klient/src/main/java/no/nav/vedtak/felles/integrasjon/dokarkiv/dto/Dokumentvariant.java @@ -1,5 +1,6 @@ package no.nav.vedtak.felles.integrasjon.dokarkiv.dto; +import java.util.Arrays; import java.util.Objects; import com.fasterxml.jackson.databind.annotation.JsonSerialize; @@ -18,15 +19,9 @@ public enum Variantformat { PRODUKSJON_DLF, SLADDET, ORIGINAL; - - public Builder medDokument(byte[] dokumentInnhold, Builder builder) { - builder.fysiskDokument = dokumentInnhold; - return builder; - } } public enum Filtype { - PDF, PDFA, XML, @@ -41,41 +36,24 @@ public enum Filtype { } - public static Builder builder() { - return new Builder(); + @Override + public boolean equals(Object o) { + if (this == o) return true; + return o instanceof Dokumentvariant that && variantformat == that.variantformat && filtype == that.filtype && Arrays.equals(fysiskDokument, that.fysiskDokument); } - public static class Builder { - - private Variantformat variantformat; - private Filtype filtype; - private byte[] fysiskDokument; - - private Builder() { - } - - public Builder medFiltype(Filtype filtype) { - this.filtype = filtype; - return this; - } - - public Builder medVariantformat(Variantformat variantformat) { - this.variantformat = variantformat; - return this; - } - - public Builder medDokument(byte[] dokumentInnhold) { - this.fysiskDokument = dokumentInnhold; - return this; - } - - public Dokumentvariant build() { - Objects.requireNonNull(fysiskDokument, "mangler dokumentinnhold"); - Objects.requireNonNull(filtype, "mangler filtype"); - Objects.requireNonNull(variantformat, "mangler variantformat"); - return new Dokumentvariant(variantformat, filtype, fysiskDokument); - } + @Override + public int hashCode() { + int result = Objects.hash(variantformat, filtype); + result = 31 * result + Arrays.hashCode(fysiskDokument); + return result; } - + @Override + public String toString() { + return "Dokumentvariant{" + + "variantformat=" + variantformat + + ", filtype=" + filtype + + '}'; + } } diff --git a/integrasjon/pdl-klient/pom.xml b/integrasjon/pdl-klient/pom.xml deleted file mode 100644 index dc8370573..000000000 --- a/integrasjon/pdl-klient/pom.xml +++ /dev/null @@ -1,26 +0,0 @@ - - - 4.0.0 - - - no.nav.foreldrepenger.felles.integrasjon - felles-integrasjon-pom - 0.0.0-SNAPSHOT - - - pdl-klient - jar - Felles :: Integrasjonsendepunkt - pdl-klient - - - - no.nav.foreldrepenger.felles.integrasjon - felles-integrasjon-rest-klient - - - no.nav.foreldrepenger.felles.integrasjon - person-klient - - - diff --git a/integrasjon/pdl-klient/src/main/java/no/nav/vedtak/felles/integrasjon/pdl/PdlKlient.java b/integrasjon/pdl-klient/src/main/java/no/nav/vedtak/felles/integrasjon/pdl/PdlKlient.java deleted file mode 100644 index 982b30db0..000000000 --- a/integrasjon/pdl-klient/src/main/java/no/nav/vedtak/felles/integrasjon/pdl/PdlKlient.java +++ /dev/null @@ -1,24 +0,0 @@ -package no.nav.vedtak.felles.integrasjon.pdl; - -import javax.enterprise.context.Dependent; - -import no.nav.vedtak.felles.integrasjon.person.AbstractPersonKlient; -import no.nav.vedtak.felles.integrasjon.person.Tema; -import no.nav.vedtak.felles.integrasjon.rest.RestClient; -import no.nav.vedtak.felles.integrasjon.rest.RestClientConfig; -import no.nav.vedtak.felles.integrasjon.rest.TokenFlow; - -@RestClientConfig(tokenConfig = TokenFlow.ADAPTIVE_ADD_CONSUMER, endpointProperty = "pdl.base.url", endpointDefault = "http://pdl-api.pdl/graphql", - scopesProperty = "pdl.scopes", scopesDefault = "api://prod-fss.pdl.pdl-api/.default") -@Dependent -public class PdlKlient extends AbstractPersonKlient { - - public PdlKlient() { - super(RestClient.client(), Tema.FOR); - } - - public PdlKlient(Tema tema) { - super(RestClient.client(), tema); - } - -} diff --git a/integrasjon/pdl-klient/src/main/java/no/nav/vedtak/felles/integrasjon/pdl/RestSupportClientProducer.java b/integrasjon/pdl-klient/src/main/java/no/nav/vedtak/felles/integrasjon/pdl/RestSupportClientProducer.java deleted file mode 100644 index b4b1e9eb8..000000000 --- a/integrasjon/pdl-klient/src/main/java/no/nav/vedtak/felles/integrasjon/pdl/RestSupportClientProducer.java +++ /dev/null @@ -1,22 +0,0 @@ -package no.nav.vedtak.felles.integrasjon.pdl; - -import javax.enterprise.context.ApplicationScoped; -import javax.enterprise.inject.Produces; - -import no.nav.vedtak.felles.integrasjon.rest.RestClient; -import no.nav.vedtak.klient.http.DefaultHttpClient; - -@ApplicationScoped -public class RestSupportClientProducer { - - @Produces - public RestClient getRestClient() { - return RestClient.client(); - } - - @Produces - public DefaultHttpClient getHttpKlient() { - return DefaultHttpClient.client(); - } - -} diff --git a/integrasjon/pdl-klient/src/main/resources/META-INF/beans.xml b/integrasjon/pdl-klient/src/main/resources/META-INF/beans.xml deleted file mode 100644 index d2cbd2f08..000000000 --- a/integrasjon/pdl-klient/src/main/resources/META-INF/beans.xml +++ /dev/null @@ -1,6 +0,0 @@ - - - diff --git a/integrasjon/pom.xml b/integrasjon/pom.xml index c8c57b46e..395f8cfb3 100644 --- a/integrasjon/pom.xml +++ b/integrasjon/pom.xml @@ -22,7 +22,6 @@ dokarkiv-klient person-klient oppgave-rest-klient - pdl-klient ereg-klient saf-klient spokelse-klient @@ -79,11 +78,6 @@ felles-integrasjon-rest-klient ${project.version} - - no.nav.foreldrepenger.felles.integrasjon - person-klient - ${project.version} - no.nav.foreldrepenger.felles.integrasjon felles-integrasjon-webservice