Skip to content

Commit ab378ca

Browse files
committed
TFP-5952 enable kall til fptilgang
1 parent ff259f9 commit ab378ca

File tree

5 files changed

+27
-15
lines changed

5 files changed

+27
-15
lines changed

felles/abac-kontekst/src/main/java/no/nav/foreldrepenger/sikkerhet/populasjon/TilgangKlient.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -45,14 +45,14 @@ public TilgangKlient() {
4545
@Override
4646
public Tilgangsvurdering vurderTilgang(PopulasjonInternRequest request) {
4747
var rrequest = RestRequest.newPOSTJson(request, internBrukerUri, restConfig);
48-
//return klient.send(rrequest, Tilgangsvurdering.class);
49-
return Tilgangsvurdering.godkjenn();
48+
return klient.send(rrequest, Tilgangsvurdering.class);
49+
//return Tilgangsvurdering.godkjenn();
5050
}
5151

5252
@Override
5353
public Tilgangsvurdering vurderTilgang(PopulasjonEksternRequest request) {
5454
var rrequest = RestRequest.newPOSTJson(request, eksternBrukerUri, restConfig);
55-
//return klient.send(rrequest, Tilgangsvurdering.class);
56-
return Tilgangsvurdering.godkjenn();
55+
return klient.send(rrequest, Tilgangsvurdering.class);
56+
//return Tilgangsvurdering.godkjenn();
5757
}
5858
}

felles/abac/src/main/java/no/nav/vedtak/sikkerhet/abac/PepImpl.java

+3-2
Original file line numberDiff line numberDiff line change
@@ -78,10 +78,11 @@ protected Tilgangsbeslutning vurderLokalTilgang(BeskyttetRessursAttributter besk
7878
private void sammenlignOgLogg(BeskyttetRessursAttributter beskyttetRessursAttributter, AppRessursData appRessursData, AbacResultat resultat) {
7979
try {
8080
var lokalt = forespørTilgang(beskyttetRessursAttributter, appRessursData);
81-
if (Objects.equals(lokalt.abacResultat(), resultat)) {
81+
if (Objects.equals(lokalt.tilgangResultat(), resultat)) {
8282
LOG.info("FPEGENTILGANG: samme svar");
8383
} else {
84-
LOG.info("FPEGENTILGANG: ulikt svar - gammel {} ny {}", resultat, lokalt.abacResultat());
84+
var metode = beskyttetRessursAttributter.getServicePath();
85+
LOG.info("FPEGENTILGANG: ulikt svar - gammel {} ny {} - metode {}", resultat, lokalt.tilgangResultat(), metode);
8586
}
8687
} catch (Exception e) {
8788
LOG.info("FPEGENTILGANG: feil", e);

felles/abac/src/main/java/no/nav/vedtak/sikkerhet/abac/policy/Tilgangsvurdering.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,10 @@
55
import no.nav.vedtak.sikkerhet.abac.AbacResultat;
66
import no.nav.vedtak.sikkerhet.kontekst.AnsattGruppe;
77

8-
public record Tilgangsvurdering(AbacResultat abacResultat, String årsak, Set<AnsattGruppe> kreverGrupper) {
8+
public record Tilgangsvurdering(AbacResultat tilgangResultat, String årsak, Set<AnsattGruppe> kreverGrupper) {
99

1010
public boolean fikkTilgang() {
11-
return abacResultat == AbacResultat.GODKJENT;
11+
return tilgangResultat == AbacResultat.GODKJENT;
1212
}
1313

1414
public static Tilgangsvurdering godkjenn() {

integrasjon/pip-klient/src/main/java/no/nav/vedtak/felles/integrasjon/skjermingpip/AbstractSkjermetPersonPipKlient.java

+15-7
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import java.net.URI;
44
import java.util.List;
55
import java.util.Map;
6+
import java.util.Objects;
67

78
import jakarta.ws.rs.core.UriBuilder;
89

@@ -70,26 +71,33 @@ private boolean kallMedSjekk(RestRequest request) {
7071
}
7172

7273
@Override
73-
public boolean erNoenSkjermet(List<String> fnr) {
74+
public Map<String, Boolean> erSkjermet(List<String> fnr) {
7475
if (fnr == null || fnr.isEmpty()) {
75-
return false;
76+
return Map.of();
7677
}
7778

7879
var request = RestRequest.newPOSTJson(new SkjermetBulkRequestDto(fnr), bulkEndpoint, restConfig);
7980

8081
try {
81-
return kallBulkMedSjekk(request);
82+
return kallBulk(request);
8283
} catch (Exception e) {
8384
LOG.info("SkjermetPerson fikk feil", e);
8485
}
85-
return kallBulkMedSjekk(request);
86+
return kallBulk(request);
87+
}
88+
89+
@Override
90+
public boolean erNoenSkjermet(List<String> fnr) {
91+
if (fnr == null || fnr.isEmpty()) {
92+
return false;
93+
}
94+
return erSkjermet(fnr).values().stream().filter(Objects::nonNull).anyMatch(v -> v);
8695
}
8796

8897
@SuppressWarnings("unchecked")
89-
private boolean kallBulkMedSjekk(RestRequest request) {
98+
private Map<String, Boolean> kallBulk(RestRequest request) {
9099
// Se github / skjerming / PipController
91-
Map<String, Boolean> skjermet = client.send(request, Map.class);
92-
return skjermet.values().stream().anyMatch(v -> v);
100+
return client.send(request, Map.class);
93101
}
94102

95103
private record SkjermetRequestDto(String personident) { }
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,14 @@
11
package no.nav.vedtak.felles.integrasjon.skjermingpip;
22

33
import java.util.List;
4+
import java.util.Map;
45

56
public interface SkjermetPersonPip {
67

78
boolean erSkjermet(String fnr);
89

10+
Map<String, Boolean> erSkjermet(List<String> fnr);
11+
912
boolean erNoenSkjermet(List<String> fnr);
1013

1114
}

0 commit comments

Comments
 (0)