Skip to content

Commit 6f25cdd

Browse files
authoredFeb 12, 2025··
TFP-5952 fjerne ubrukt legacy-abac (#1415)
1 parent 0932ad0 commit 6f25cdd

36 files changed

+26
-1368
lines changed
 

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/abac/BeskyttetRessursInterceptor.java

-4
Original file line numberDiff line numberDiff line change
@@ -60,19 +60,15 @@ private Object ikkeTilgang(AbacResultat abacResultat) {
6060
private BeskyttetRessursAttributter hentBeskyttetRessursAttributter(Method method, Class<?> mClass, AbacDataAttributter dataAttributter) {
6161
var beskyttetRessurs = method.getAnnotation(BeskyttetRessurs.class);
6262

63-
var token = Token.withOidcToken(tokenProvider.openIdToken());
64-
6563
return BeskyttetRessursAttributter.builder()
6664
.medBrukerId(tokenProvider.getUid())
6765
.medBrukerOid(tokenProvider.getOid())
6866
.medIdentType(tokenProvider.getIdentType())
6967
.medAnsattGrupper(tokenProvider.getAnsattGrupper())
70-
.medToken(token)
7168
.medActionType(beskyttetRessurs.actionType())
7269
.medAvailabilityType(beskyttetRessurs.availabilityType())
7370
.medResourceType(finnResource(beskyttetRessurs))
7471
.medSporingslogg(beskyttetRessurs.sporingslogg())
75-
.medPepId(pep.pepId())
7672
.medServicePath(utledAction(mClass, method))
7773
.medDataAttributter(dataAttributter)
7874
.build();

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/abac/PdpKlient.java

-10
This file was deleted.
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,9 @@
11
package no.nav.vedtak.sikkerhet.abac;
22

3-
import no.nav.foreldrepenger.konfig.Environment;
43
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursAttributter;
54

65
public interface Pep {
76

87
AbacResultat vurderTilgang(BeskyttetRessursAttributter beskyttetRessursAttributter);
98

10-
default String pepId() {
11-
return Environment.current().getNaisAppName();
12-
}
139
}

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/abac/Token.java

-70
This file was deleted.
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,14 @@
11
package no.nav.vedtak.sikkerhet.abac.beskyttet;
22

33
public enum ActionType {
4-
READ("read"),
5-
UPDATE("update"),
6-
CREATE("create"),
7-
DELETE("delete"),
4+
READ,
5+
UPDATE,
6+
CREATE,
7+
DELETE,
88

99
/**
1010
* Skal kun brukes av Interceptor
1111
*/
12-
DUMMY(null);
12+
DUMMY;
1313

14-
private String eksternKode;
15-
16-
ActionType(String eksternKode) {
17-
this.eksternKode = eksternKode;
18-
}
19-
20-
public String getEksternKode() {
21-
return eksternKode;
22-
}
2314
}
Original file line numberDiff line numberDiff line change
@@ -1,44 +1,25 @@
11
package no.nav.vedtak.sikkerhet.abac.beskyttet;
22

3-
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_APPLIKASJON;
4-
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_AVDELINGENHET;
5-
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_DRIFT;
6-
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_FAGSAK;
7-
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_OPPGAVESTYRING;
8-
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_UTTAKSPLAN;
9-
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_FP_VENTEFRIST;
10-
import static no.nav.vedtak.sikkerhet.abac.policy.ForeldrepengerAttributter.RESOURCE_TYPE_INTERNAL_PIP;
11-
123
public enum ResourceType {
134

145
// Til bruk i annotering
15-
APPLIKASJON(RESOURCE_TYPE_FP_APPLIKASJON),
16-
DRIFT(RESOURCE_TYPE_FP_DRIFT),
17-
FAGSAK(RESOURCE_TYPE_FP_FAGSAK),
18-
VENTEFRIST(RESOURCE_TYPE_FP_VENTEFRIST),
6+
APPLIKASJON,
7+
DRIFT,
8+
FAGSAK,
9+
VENTEFRIST,
1910
// LOS
20-
OPPGAVESTYRING_AVDELINGENHET(RESOURCE_TYPE_FP_AVDELINGENHET),
21-
OPPGAVESTYRING(RESOURCE_TYPE_FP_OPPGAVESTYRING),
22-
// OPPGAVEKØ(RESOURCE_TYPE_FP_OPPGAVEKØ), TODO: Vurder om skal brukes for å lese oppgaver for LOS. Nå brukes FAGSAK
11+
OPPGAVESTYRING_AVDELINGENHET,
12+
OPPGAVESTYRING,
13+
// OPPGAVEKØ, TODO: Vurder om skal brukes for å lese oppgaver for LOS. Nå brukes FAGSAK
2314

2415
// Selvbetjening
25-
UTTAKSPLAN(RESOURCE_TYPE_FP_UTTAKSPLAN),
16+
UTTAKSPLAN,
2617

2718
// Til bruk i annotering for endepunkt som er PIP-tjenester
28-
PIP(RESOURCE_TYPE_INTERNAL_PIP),
19+
PIP,
2920

3021
/**
3122
* Skal kun brukes av Interceptor
3223
*/
33-
DUMMY("");
34-
35-
private final String resourceTypeAttribute;
36-
37-
ResourceType(String resourceTypeAttribute) {
38-
this.resourceTypeAttribute = resourceTypeAttribute;
39-
}
40-
41-
public String getResourceTypeAttribute() {
42-
return this != DUMMY ? resourceTypeAttribute : null;
43-
}
24+
DUMMY
4425
}

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/abac/internal/BeskyttetRessursAttributter.java

+2-24
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
import java.util.UUID;
77

88
import no.nav.vedtak.sikkerhet.abac.AbacDataAttributter;
9-
import no.nav.vedtak.sikkerhet.abac.Token;
109
import no.nav.vedtak.sikkerhet.abac.beskyttet.ActionType;
1110
import no.nav.vedtak.sikkerhet.abac.beskyttet.AvailabilityType;
1211
import no.nav.vedtak.sikkerhet.abac.beskyttet.ResourceType;
@@ -23,8 +22,6 @@ public class BeskyttetRessursAttributter {
2322
private ActionType actionType;
2423
private ResourceType resourceType;
2524
private AvailabilityType availabilityType;
26-
private Token token;
27-
private String pepId;
2825
private String servicePath;
2926
private boolean sporingslogg = true;
3027
private AbacDataAttributter dataAttributter;
@@ -65,14 +62,6 @@ public boolean isSporingslogg() {
6562
return sporingslogg;
6663
}
6764

68-
public Token getToken() {
69-
return token;
70-
}
71-
72-
public String getPepId() {
73-
return pepId;
74-
}
75-
7665
public String getServicePath() {
7766
return servicePath;
7867
}
@@ -84,7 +73,7 @@ public AbacDataAttributter getDataAttributter() {
8473
@Override
8574
public String toString() {
8675
return "BeskyttetRessursAttributter{" + "userId=MASKERT" + ", actionType=" + actionType + ", resourceType="
87-
+ resourceType + ", token=" + token + ", pepId=" + pepId + ", servicePath=" + servicePath + '}';
76+
+ resourceType + ", servicePath=" + servicePath + '}';
8877
}
8978

9079
public static class Builder {
@@ -114,11 +103,6 @@ public Builder medAnsattGrupper(Set<AnsattGruppe> ansattGrupper) {
114103
return this;
115104
}
116105

117-
public Builder medToken(Token token) {
118-
pdpRequest.token = token;
119-
return this;
120-
}
121-
122106
public Builder medActionType(ActionType actionType) {
123107
pdpRequest.actionType = actionType;
124108
return this;
@@ -139,11 +123,6 @@ public Builder medSporingslogg(boolean sporingslogg) {
139123
return this;
140124
}
141125

142-
public Builder medPepId(String pepId) {
143-
pdpRequest.pepId = pepId;
144-
return this;
145-
}
146-
147126
public Builder medServicePath(String servicePath) {
148127
pdpRequest.servicePath = servicePath;
149128
return this;
@@ -161,10 +140,9 @@ public BeskyttetRessursAttributter build() {
161140

162141
private void validateBeforeBuild() {
163142
Objects.requireNonNull(pdpRequest.brukerId, "userId");
164-
Objects.requireNonNull(pdpRequest.token, "idToken");
165143
Objects.requireNonNull(pdpRequest.actionType, "actionType");
166144
Objects.requireNonNull(pdpRequest.resourceType, "resourceType");
167-
Objects.requireNonNull(pdpRequest.pepId, "pepId");
145+
Objects.requireNonNull(pdpRequest.identType, "identType");
168146
Objects.requireNonNull(pdpRequest.servicePath, "servicePath");
169147
Objects.requireNonNull(pdpRequest.dataAttributter, "dataAttributter");
170148
}

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

-20
Original file line numberDiff line numberDiff line change
@@ -38,24 +38,4 @@ private ForeldrepengerAttributter() {
3838
public static final String VALUE_FP_AVDELING_ENHET_ADRESSEBESKYTTET = "2103";
3939
public static final String VALUE_FP_AVDELING_ENHET_SKJERMET = "4883";
4040

41-
/**
42-
* Attributter brukt som resource_type
43-
* TODO: Behov for AvdelingEnhet og OppgaveStyring? Fjerne risikoklassifisering?
44-
* TODO: OPPGAVEKØ ikke i bruk - vurder FAGSAK vs OPPGAVEKØ + evt bruk som dataattributt (køer på 2103. Mangler policies)
45-
*/
46-
public static final String RESOURCE_TYPE_FP_APPLIKASJON = "no.nav.abac.attributter.foreldrepenger";
47-
public static final String RESOURCE_TYPE_FP_DRIFT = "no.nav.abac.attributter.foreldrepenger.drift";
48-
public static final String RESOURCE_TYPE_FP_FAGSAK = "no.nav.abac.attributter.foreldrepenger.fagsak";
49-
public static final String RESOURCE_TYPE_FP_VENTEFRIST = "no.nav.abac.attributter.foreldrepenger.fagsak.ventefrist";
50-
public static final String RESOURCE_TYPE_FP_AVDELINGENHET = "no.nav.abac.attributter.foreldrepenger.oppgavestyring.avdelingsenhet";
51-
// public static final String RESOURCE_TYPE_FP_OPPGAVEKØ = "no.nav.abac.attributter.foreldrepenger.oppgaveko"; TODO: Vurder om skal brukes for å lese oppgaver for LOS. Nå brukes FAGSAK. Evt bruk som dataAttributt.
52-
public static final String RESOURCE_TYPE_FP_OPPGAVESTYRING = "no.nav.abac.attributter.foreldrepenger.oppgavestyring";
53-
public static final String RESOURCE_TYPE_FP_UTTAKSPLAN = "no.nav.abac.attributter.resource.foreldrepenger.uttaksplan";
54-
55-
/**
56-
* Attributter brukt til interne formål
57-
*/
58-
public static final String RESOURCE_TYPE_INTERNAL_PIP = "pip.tjeneste.kan.kun.kalles.av.pdp.servicebruker";
59-
public static final String RESOURCE_TYPE_INTERNAL_DUMMY = "";
60-
6141
}

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

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package no.nav.vedtak.sikkerhet.abac.policy;
22

33
import java.util.Objects;
4+
import java.util.Optional;
45
import java.util.Set;
56

67
import no.nav.foreldrepenger.konfig.Cluster;
@@ -25,7 +26,8 @@ public class SystemressursPolicies {
2526
private static final Environment ENV = Environment.current();
2627

2728
// Format: json array av objekt("name", "clientId");
28-
private static final String PRE_AUTHORIZED = ENV.getProperty(AzureProperty.AZURE_APP_PRE_AUTHORIZED_APPS.name());
29+
private static final String PRE_AUTHORIZED = Optional.ofNullable(ENV.getProperty(AzureProperty.AZURE_APP_PRE_AUTHORIZED_APPS.name()))
30+
.orElseGet(() -> ENV.getProperty(AzureProperty.AZURE_APP_PRE_AUTHORIZED_APPS.name().toLowerCase().replace('_', '.')));
2931
private static final Cluster RESIDENT_CLUSTER = ENV.getCluster();
3032
private static final String RESIDENT_NAMESPACE = ENV.namespace();
3133
private static final Set<ResourceType> IKKE_TILLATT_RESOURCE_TYPE = Set.of(ResourceType.UTTAKSPLAN);

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/PdpConsumer.java

-9
This file was deleted.

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/PdpConsumerImpl.java

-102
This file was deleted.

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/PdpKlientImpl.java

-92
This file was deleted.

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/XacmlRequestMapper.java

-105
This file was deleted.

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/xacml/Advice.java

-8
This file was deleted.

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/xacml/Category.java

-12
This file was deleted.

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/xacml/Decision.java

-8
This file was deleted.

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/xacml/NavFellesAttributter.java

-23
This file was deleted.

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/xacml/XacmlRequest.java

-18
This file was deleted.

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/xacml/XacmlResponse.java

-22
This file was deleted.

‎felles/abac/src/main/java/no/nav/vedtak/sikkerhet/pdp/xacml/XacmlResponseMapper.java

-66
This file was deleted.

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/abac/AbacAuditLoggerTest.java

-3
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@
2525
import no.nav.vedtak.sikkerhet.abac.beskyttet.ResourceType;
2626
import no.nav.vedtak.sikkerhet.abac.internal.ActionUthenter;
2727
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursAttributter;
28-
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursInterceptorTest;
2928
import no.nav.vedtak.sikkerhet.abac.pdp.AppRessursData;
3029
import no.nav.vedtak.sikkerhet.kontekst.IdentType;
3130

@@ -121,12 +120,10 @@ private BeskyttetRessursAttributter getBeskyttetRessursAttributter(Method method
121120
.medBrukerOid(UUID.randomUUID())
122121
.medIdentType(IdentType.InternBruker)
123122
.medAnsattGrupper(Set.of())
124-
.medToken(Token.withOidcToken(BeskyttetRessursInterceptorTest.DUMMY_OPENID_TOKEN))
125123
.medActionType(beskyttetRessurs.actionType())
126124
.medAvailabilityType(beskyttetRessurs.availabilityType())
127125
.medResourceType(beskyttetRessurs.resourceType())
128126
.medSporingslogg(beskyttetRessurs.sporingslogg())
129-
.medPepId("local-app")
130127
.medServicePath(ActionUthenter.action(RestClass.class, method))
131128
.medDataAttributter(dataAttributter)
132129
.build();

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/abac/PepImplTest.java

+5-21
Original file line numberDiff line numberDiff line change
@@ -22,15 +22,11 @@
2222
import no.nav.vedtak.sikkerhet.abac.beskyttet.AvailabilityType;
2323
import no.nav.vedtak.sikkerhet.abac.beskyttet.ResourceType;
2424
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursAttributter;
25-
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursInterceptorTest;
2625
import no.nav.vedtak.sikkerhet.abac.pdp.AppRessursData;
2726
import no.nav.vedtak.sikkerhet.abac.policy.Tilgangsvurdering;
2827
import no.nav.vedtak.sikkerhet.kontekst.AnsattGruppe;
2928
import no.nav.vedtak.sikkerhet.kontekst.IdentType;
3029
import no.nav.vedtak.sikkerhet.oidc.config.AzureProperty;
31-
import no.nav.vedtak.sikkerhet.oidc.config.OpenIDProvider;
32-
import no.nav.vedtak.sikkerhet.oidc.token.OpenIDToken;
33-
import no.nav.vedtak.sikkerhet.oidc.token.TokenString;
3430
import no.nav.vedtak.sikkerhet.tilgang.AnsattGruppeKlient;
3531
import no.nav.vedtak.sikkerhet.tilgang.PopulasjonKlient;
3632

@@ -94,9 +90,8 @@ void skal_nekte_tilgang_til_saksbehandler_for_piptjeneste() {
9490

9591
@Test
9692
void skal_gi_tilgang_for_intern_azure_cc() {
97-
var token = new OpenIDToken(OpenIDProvider.AZUREAD, new TokenString("token"));
9893
when(tokenProvider.getUid()).thenReturn(LOCAL_APP);
99-
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.INTERNAL, token, IdentType.Systemressurs);
94+
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.INTERNAL, IdentType.Systemressurs);
10095

10196
when(pdpRequestBuilder.lagAppRessursData(any())).thenReturn(AppRessursData.builder().build());
10297

@@ -108,9 +103,8 @@ void skal_gi_tilgang_for_intern_azure_cc() {
108103

109104
@Test
110105
void skal_gi_avslag_for_ekstern_azure_cc() {
111-
var token = new OpenIDToken(OpenIDProvider.AZUREAD, new TokenString("token"));
112106
when(tokenProvider.getUid()).thenReturn("vtp:annetnamespace:ukjentapplication");
113-
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.INTERNAL, token,
107+
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.INTERNAL,
114108
IdentType.Systemressurs);
115109

116110
when(pdpRequestBuilder.lagAppRessursData(any())).thenReturn(AppRessursData.builder().build());
@@ -123,9 +117,8 @@ void skal_gi_avslag_for_ekstern_azure_cc() {
123117

124118
@Test
125119
void skal_gi_avslag_for_godkjent_ekstern_azure_cc_men_i_feil_klusterklasse() {
126-
var token = new OpenIDToken(OpenIDProvider.AZUREAD, new TokenString("token"));
127120
when(tokenProvider.getUid()).thenReturn("dev-fss:annetnamespace:eksternapplication");
128-
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.INTERNAL, token,
121+
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.INTERNAL,
129122
IdentType.Systemressurs);
130123

131124
when(pdpRequestBuilder.lagAppRessursData(any())).thenReturn(AppRessursData.builder().build());
@@ -139,9 +132,8 @@ void skal_gi_avslag_for_godkjent_ekstern_azure_cc_men_i_feil_klusterklasse() {
139132

140133
@Test
141134
void skal_gi_tilgang_for_godkjent_ekstern_azure_cc() {
142-
var token = new OpenIDToken(OpenIDProvider.AZUREAD, new TokenString("token"));
143135
when(tokenProvider.getUid()).thenReturn("vtp:annetnamespace:eksternapplication");
144-
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.ALL, token,
136+
var attributter = lagBeskyttetRessursAttributterAzure(AvailabilityType.ALL,
145137
IdentType.Systemressurs);
146138

147139
when(pdpRequestBuilder.lagAppRessursData(any())).thenReturn(AppRessursData.builder().build());
@@ -172,10 +164,8 @@ private BeskyttetRessursAttributter lagBeskyttetRessursAttributter() {
172164
.medBrukerOid(UUID.randomUUID())
173165
.medIdentType(IdentType.InternBruker)
174166
.medAnsattGrupper(Set.of(AnsattGruppe.SAKSBEHANDLER))
175-
.medToken(Token.withOidcToken(BeskyttetRessursInterceptorTest.DUMMY_OPENID_TOKEN))
176167
.medResourceType(ResourceType.FAGSAK)
177168
.medActionType(ActionType.READ)
178-
.medPepId("local-app")
179169
.medServicePath("/metode")
180170
.medDataAttributter(AbacDataAttributter.opprett())
181171
.build();
@@ -185,26 +175,20 @@ private BeskyttetRessursAttributter lagBeskyttetRessursAttributterPip() {
185175
return BeskyttetRessursAttributter.builder()
186176
.medBrukerId(tokenProvider.getUid())
187177
.medIdentType(IdentType.InternBruker)
188-
.medToken(Token.withOidcToken(BeskyttetRessursInterceptorTest.DUMMY_OPENID_TOKEN))
189178
.medResourceType(ResourceType.PIP)
190179
.medActionType(ActionType.READ)
191-
.medPepId("local-app")
192180
.medServicePath("/metode")
193181
.medDataAttributter(AbacDataAttributter.opprett())
194182
.build();
195183
}
196184

197-
private BeskyttetRessursAttributter lagBeskyttetRessursAttributterAzure(AvailabilityType availabilityType,
198-
OpenIDToken token,
199-
IdentType identType) {
185+
private BeskyttetRessursAttributter lagBeskyttetRessursAttributterAzure(AvailabilityType availabilityType, IdentType identType) {
200186
return BeskyttetRessursAttributter.builder()
201187
.medBrukerId(tokenProvider.getUid())
202188
.medIdentType(identType)
203-
.medToken(Token.withOidcToken(token))
204189
.medResourceType(ResourceType.FAGSAK)
205190
.medActionType(ActionType.READ)
206191
.medAvailabilityType(availabilityType)
207-
.medPepId("local-app")
208192
.medServicePath("/metode")
209193
.medDataAttributter(AbacDataAttributter.opprett())
210194
.build();

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/abac/internal/BeskyttetRessursInterceptorTest.java

+1-12
Original file line numberDiff line numberDiff line change
@@ -33,27 +33,20 @@
3333
import no.nav.vedtak.sikkerhet.abac.beskyttet.AvailabilityType;
3434
import no.nav.vedtak.sikkerhet.abac.beskyttet.ResourceType;
3535
import no.nav.vedtak.sikkerhet.kontekst.IdentType;
36-
import no.nav.vedtak.sikkerhet.oidc.config.OpenIDProvider;
37-
import no.nav.vedtak.sikkerhet.oidc.token.OpenIDToken;
38-
import no.nav.vedtak.sikkerhet.oidc.token.TokenString;
3936

4037
@ExtendWith(MockitoExtension.class)
41-
public class BeskyttetRessursInterceptorTest {
38+
class BeskyttetRessursInterceptorTest {
4239

43-
private static final String DUMMY_ID_TOKEN = "eyJraWQiOiI3Mzk2ZGIyZC1hN2MyLTQ1OGEtYjkzNC02ODNiNDgzYzUyNDIiLCJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiJ9.eyJhdF9oYXNoIjoiRzJ1Zl83OW1TTUhHSWFfNjFxTnJfUSIsInN1YiI6IjA5MDg4NDIwNjcyIiwidmVyIjoiMS4wIiwiaXNzIjoiaHR0cHM6XC9cL3Rva2VuZGluZ3MuZGV2LWdjcC5uYWlzLmlvIiwibm9uY2UiOiJWR1dyS1Zsa3RXZ3hCdTlMZnNnMHliMmdMUVhoOHRaZHRaVTJBdWdPZVl3IiwiY2xpZW50X2lkIjoiZGV2LWZzczp0ZWFtZm9yZWxkcmVwZW5nZXI6ZnBzb2tuYWQtbW90dGFrIiwiYXVkIjoiZGV2LWZzczp0ZWFtZm9yZWxkcmVwZW5nZXI6ZnBpbmZvIiwiYWNyIjoiTGV2ZWw0IiwibmJmIjoxNjE2Njg1NDA0LCJpZHAiOiJodHRwczpcL1wvbmF2dGVzdGIyYy5iMmNsb2dpbi5jb21cL2QzOGYyNWFhLWVhYjgtNGM1MC05ZjI4LWViZjkyYzEyNTZmMlwvdjIuMFwvIiwiYXV0aF90aW1lIjoxNjE2Njg1NDAyLCJleHAiOjE2MTY2ODU3MDQsImlhdCI6MTYxNjY4NTQwNCwianRpIjoiNGMwNzBmMGUtNzI0Ny00ZTdjLWE1OWEtYzk2Yjk0NWMxZWZhIn0.OvzjuabvPHG9nlRVc_KlCUTHOdfeT9GtBkASUGIoMayWGeIBDkr4-jc9gu6uT_WQqi9IJnvPkWgP3veqYHcOHpapD1yVNaQpxlrJQ04yP6N3gvkn-DcrBRDb3II_6qSaPQ_us2PJBDPq2VD5TGrNOL6EFwr8FK3zglYr-PgjW016ULTcmx_7gdHmbiC5PEn1_OtGNxzoUhSGKoD3YtUWP0qdsXzoKyeFL5FG9uZMSrDHHiJBZQFXGL9OzBU49Zb2K-iEPqa9m91O2JZGkhebfLjCAIPLPN4J68GFyfTvtNkZO71znorjo-e1nWxz53Wkj---RDY3JlIqNqzqHTfJgQ";
4440
private final RestClass tjeneste = new RestClass();
4541

4642
private final AktørDto aktør1 = new AktørDto("00000000000");
4743
private final BehandlingIdDto behandlingIdDto = new BehandlingIdDto(1234L);
4844

4945
private static final String BRUKER_IDENT = "A000000";
5046
private static final UUID BRUKER_OID = UUID.randomUUID();
51-
private static final String PEP_ID = "test";
5247

5348
private final ArgumentCaptor<BeskyttetRessursAttributter> braCaptor = ArgumentCaptor.forClass(BeskyttetRessursAttributter.class);
5449

55-
public static final OpenIDToken DUMMY_OPENID_TOKEN = new OpenIDToken(OpenIDProvider.TOKENX, new TokenString(DUMMY_ID_TOKEN));
56-
5750
@Mock
5851
private TokenProvider tokenProvider;
5952
@Mock
@@ -66,8 +59,6 @@ void mockTokenProvider() {
6659
when(tokenProvider.getOid()).thenReturn(BRUKER_OID);
6760
when(tokenProvider.getAnsattGrupper()).thenReturn(Set.of());
6861
when(tokenProvider.getIdentType()).thenReturn(IdentType.InternBruker);
69-
when(tokenProvider.openIdToken()).thenReturn(DUMMY_OPENID_TOKEN);
70-
when(pep.pepId()).thenReturn(PEP_ID);
7162
}
7263

7364
@Test
@@ -135,8 +126,6 @@ private void assertBeskyttetRessursAttributter(BeskyttetRessursAttributter bra)
135126
assertThat(bra.getActionType()).isEqualTo(ActionType.CREATE);
136127
assertThat(bra.getResourceType()).isEqualTo(ResourceType.PIP);
137128
assertThat(bra.getAvailabilityType()).isEqualTo(AvailabilityType.INTERNAL);
138-
assertThat(bra.getPepId()).isEqualTo(PEP_ID);
139-
assertThat(bra.getToken().getOpenIDProvider()).isEqualTo(OpenIDProvider.TOKENX);
140129
assertThat(bra.getServicePath()).startsWith("/foo");
141130
assertThat(bra.getDataAttributter().keySet()).hasSize(1);
142131
}

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/abac/policy/EksternBrukerTest.java

-4
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,9 @@
77
import org.mockito.junit.jupiter.MockitoExtension;
88

99
import no.nav.vedtak.sikkerhet.abac.AbacDataAttributter;
10-
import no.nav.vedtak.sikkerhet.abac.Token;
1110
import no.nav.vedtak.sikkerhet.abac.beskyttet.ActionType;
1211
import no.nav.vedtak.sikkerhet.abac.beskyttet.ResourceType;
1312
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursAttributter;
14-
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursInterceptorTest;
1513
import no.nav.vedtak.sikkerhet.abac.pdp.AppRessursData;
1614
import no.nav.vedtak.sikkerhet.kontekst.IdentType;
1715

@@ -97,10 +95,8 @@ private BeskyttetRessursAttributter lagAttributter(ActionType actionType, Resour
9795
return BeskyttetRessursAttributter.builder()
9896
.medBrukerId("12345678901")
9997
.medIdentType(IdentType.EksternBruker)
100-
.medToken(Token.withOidcToken(BeskyttetRessursInterceptorTest.DUMMY_OPENID_TOKEN))
10198
.medResourceType(resourceType)
10299
.medActionType(actionType)
103-
.medPepId("local-app")
104100
.medServicePath("/metode")
105101
.medDataAttributter(AbacDataAttributter.opprett())
106102
.build();

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/abac/policy/InternBrukerFagsakTest.java

-4
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,9 @@
1010
import org.mockito.junit.jupiter.MockitoExtension;
1111

1212
import no.nav.vedtak.sikkerhet.abac.AbacDataAttributter;
13-
import no.nav.vedtak.sikkerhet.abac.Token;
1413
import no.nav.vedtak.sikkerhet.abac.beskyttet.ActionType;
1514
import no.nav.vedtak.sikkerhet.abac.beskyttet.ResourceType;
1615
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursAttributter;
17-
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursInterceptorTest;
1816
import no.nav.vedtak.sikkerhet.abac.pdp.AppRessursData;
1917
import no.nav.vedtak.sikkerhet.abac.pipdata.PipBehandlingStatus;
2018
import no.nav.vedtak.sikkerhet.abac.pipdata.PipFagsakStatus;
@@ -232,10 +230,8 @@ private BeskyttetRessursAttributter lagAttributter(AnsattGruppe ansattGruppe, Ac
232230
.medBrukerOid(UUID.randomUUID())
233231
.medIdentType(IdentType.InternBruker)
234232
.medAnsattGrupper(Set.of(ansattGruppe))
235-
.medToken(Token.withOidcToken(BeskyttetRessursInterceptorTest.DUMMY_OPENID_TOKEN))
236233
.medResourceType(ResourceType.FAGSAK)
237234
.medActionType(actionType)
238-
.medPepId("local-app")
239235
.medServicePath("/metode")
240236
.medDataAttributter(AbacDataAttributter.opprett())
241237
.build();

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/abac/policy/InternBrukerTest.java

-4
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,9 @@
1010
import org.mockito.junit.jupiter.MockitoExtension;
1111

1212
import no.nav.vedtak.sikkerhet.abac.AbacDataAttributter;
13-
import no.nav.vedtak.sikkerhet.abac.Token;
1413
import no.nav.vedtak.sikkerhet.abac.beskyttet.ActionType;
1514
import no.nav.vedtak.sikkerhet.abac.beskyttet.ResourceType;
1615
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursAttributter;
17-
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursInterceptorTest;
1816
import no.nav.vedtak.sikkerhet.abac.pdp.AppRessursData;
1917
import no.nav.vedtak.sikkerhet.abac.pdp.ForeldrepengerDataKeys;
2018
import no.nav.vedtak.sikkerhet.kontekst.AnsattGruppe;
@@ -161,10 +159,8 @@ private BeskyttetRessursAttributter lagAttributter(AnsattGruppe ansattGruppe, Ac
161159
.medBrukerOid(UUID.randomUUID())
162160
.medIdentType(IdentType.InternBruker)
163161
.medAnsattGrupper(Set.of(ansattGruppe))
164-
.medToken(Token.withOidcToken(BeskyttetRessursInterceptorTest.DUMMY_OPENID_TOKEN))
165162
.medResourceType(resourceType)
166163
.medActionType(actionType)
167-
.medPepId("local-app")
168164
.medServicePath("/metode")
169165
.medDataAttributter(AbacDataAttributter.opprett())
170166
.build();

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/abac/policy/SystemressursTest.java

-6
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,10 @@
1010

1111
import no.nav.foreldrepenger.konfig.Namespace;
1212
import no.nav.vedtak.sikkerhet.abac.AbacDataAttributter;
13-
import no.nav.vedtak.sikkerhet.abac.Token;
1413
import no.nav.vedtak.sikkerhet.abac.beskyttet.ActionType;
1514
import no.nav.vedtak.sikkerhet.abac.beskyttet.AvailabilityType;
1615
import no.nav.vedtak.sikkerhet.abac.beskyttet.ResourceType;
1716
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursAttributter;
18-
import no.nav.vedtak.sikkerhet.abac.internal.BeskyttetRessursInterceptorTest;
1917
import no.nav.vedtak.sikkerhet.abac.pdp.AppRessursData;
2018
import no.nav.vedtak.sikkerhet.abac.pipdata.PipBehandlingStatus;
2119
import no.nav.vedtak.sikkerhet.kontekst.IdentType;
@@ -92,10 +90,8 @@ private BeskyttetRessursAttributter lagBeskyttetRessursAttributter() {
9290
return BeskyttetRessursAttributter.builder()
9391
.medBrukerId(LOCAL_APP)
9492
.medIdentType(IdentType.Systemressurs)
95-
.medToken(Token.withOidcToken(BeskyttetRessursInterceptorTest.DUMMY_OPENID_TOKEN))
9693
.medResourceType(ResourceType.FAGSAK)
9794
.medActionType(ActionType.UPDATE)
98-
.medPepId("local-app")
9995
.medServicePath("/metode")
10096
.medDataAttributter(AbacDataAttributter.opprett())
10197
.build();
@@ -105,11 +101,9 @@ private BeskyttetRessursAttributter lagBeskyttetRessursAttributterAzure(Availabi
105101
return BeskyttetRessursAttributter.builder()
106102
.medBrukerId(brukerId)
107103
.medIdentType(IdentType.Systemressurs)
108-
.medToken(Token.withOidcToken(BeskyttetRessursInterceptorTest.DUMMY_OPENID_TOKEN))
109104
.medResourceType(ResourceType.APPLIKASJON)
110105
.medActionType(ActionType.READ)
111106
.medAvailabilityType(availabilityType)
112-
.medPepId("local-app")
113107
.medServicePath("/metode")
114108
.medDataAttributter(AbacDataAttributter.opprett())
115109
.build();

‎felles/abac/src/test/java/no/nav/vedtak/sikkerhet/pdp/PdpKlientImplTest.java

-302
This file was deleted.

‎felles/abac/src/test/resources/request.json

-58
This file was deleted.

‎felles/abac/src/test/resources/request1.json

-74
This file was deleted.

‎felles/abac/src/test/resources/xacml3response.json

-64
This file was deleted.

‎felles/abac/src/test/resources/xacmlresponse-array.json

-33
This file was deleted.

‎felles/abac/src/test/resources/xacmlresponse.json

-20
This file was deleted.

‎felles/abac/src/test/resources/xacmlresponse_1deny_1permit.json

-60
This file was deleted.

‎felles/abac/src/test/resources/xacmlresponse_multiple_obligation.json

-37
This file was deleted.

‎felles/kontekst/src/main/java/no/nav/vedtak/sikkerhet/kontekst/Systembruker.java

-25
This file was deleted.

0 commit comments

Comments
 (0)
Please sign in to comment.