Skip to content

Commit 1fb29c6

Browse files
committed
Fjerne all bruk av Systembruker utenom ABAC
1 parent a275c88 commit 1fb29c6

File tree

2 files changed

+7
-14
lines changed

2 files changed

+7
-14
lines changed

felles/kontekst/src/main/java/no/nav/vedtak/sikkerhet/kontekst/BasisKontekst.java

+4-11
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,6 @@ public String getKonsumentId() {
5050
return konsumentId;
5151
}
5252

53-
// Brukes kun i abakus / dignostikk. Sjekk om kan endres til forProsesstaskUtenSystembruker
54-
@Deprecated(forRemoval = true) // Erstatt med forProsesstaskUtenSystembruker
55-
public static BasisKontekst forProsesstask() {
56-
return new BasisKontekst(SikkerhetContext.SYSTEM, Systembruker.username(), IdentType.Prosess, Systembruker.username());
57-
}
58-
5953
// Denne brukes i prosesstask
6054
public static BasisKontekst forProsesstaskUtenSystembruker() {
6155
var username = "srv" + Optional.ofNullable(Environment.current().application()).orElse("local");
@@ -64,15 +58,14 @@ public static BasisKontekst forProsesstaskUtenSystembruker() {
6458
}
6559

6660
public static BasisKontekst ikkeAutentisertRequest(String consumerId) {
67-
return new BasisKontekst(SikkerhetContext.REQUEST, null, null, ensureCunsumerId(consumerId));
61+
var consumer = Optional.ofNullable(consumerId)
62+
.or(() -> Optional.ofNullable(Environment.current().application()).map(a -> "srv" + a))
63+
.orElse("srvlocal");
64+
return new BasisKontekst(SikkerhetContext.REQUEST, null, null, consumer);
6865
}
6966

7067
static BasisKontekst tomKontekst() {
7168
return new BasisKontekst(null, null, null, null);
7269
}
7370

74-
protected static String ensureCunsumerId(String consumerId) {
75-
return consumerId != null ? consumerId : Systembruker.username();
76-
}
77-
7871
}

felles/kontekst/src/test/java/no/nav/vedtak/sikkerhet/kontekst/KontekstHolderTest.java

+3-3
Original file line numberDiff line numberDiff line change
@@ -15,15 +15,15 @@ void testGetDefaultKontekst() {
1515

1616
@Test
1717
void testSetAndGetSystemKontekst() {
18-
var eksisterende = BasisKontekst.forProsesstask();
18+
var eksisterende = BasisKontekst.forProsesstaskUtenSystembruker();
1919
KontekstHolder.setKontekst(eksisterende);
2020

2121
assertThat(KontekstHolder.harKontekst()).isTrue();
2222
var roundtrip = KontekstHolder.getKontekst();
2323
assertThat(roundtrip).isNotNull();
2424
assertThat(roundtrip.getContext()).isEqualTo(SikkerhetContext.SYSTEM);
25-
assertThat(roundtrip.getUid()).isEqualTo(Systembruker.username());
26-
assertThat(roundtrip.getKompaktUid()).isEqualTo(Systembruker.username());
25+
assertThat(roundtrip.getUid()).isEqualTo("srvvtp");
26+
assertThat(roundtrip.getKompaktUid()).isEqualTo("srvvtp");
2727
assertThat(roundtrip.getIdentType()).isEqualTo(IdentType.Prosess);
2828

2929
KontekstHolder.fjernKontekst();

0 commit comments

Comments
 (0)