Skip to content

Commit 0b6bd37

Browse files
sillerudsillerud
committed
Fjern siste rester av tilgangskontroll basert på annotasjoner
1 parent 2329344 commit 0b6bd37

File tree

7 files changed

+105
-400
lines changed

7 files changed

+105
-400
lines changed

‎pom.xml

-10
Original file line numberDiff line numberDiff line change
@@ -73,16 +73,6 @@
7373
<groupId>org.springframework.boot</groupId>
7474
<artifactId>spring-boot-starter-actuator</artifactId>
7575
</dependency>
76-
<dependency>
77-
<groupId>org.aspectj</groupId>
78-
<artifactId>aspectjrt</artifactId>
79-
<version>${aspectj.version}</version>
80-
</dependency>
81-
<dependency>
82-
<groupId>org.aspectj</groupId>
83-
<artifactId>aspectjtools</artifactId>
84-
<version>${aspectj.version}</version>
85-
</dependency>
8676
<!-- Kotlin -->
8777
<dependency>
8878
<groupId>org.jetbrains.kotlin</groupId>

‎src/main/kotlin/no/nav/familie/tilbake/api/BehandlingController.kt

+20-4
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ import no.nav.familie.tilbake.kontrakter.tilbakekreving.OpprettTilbakekrevingReq
2222
import no.nav.familie.tilbake.log.SecureLog
2323
import no.nav.familie.tilbake.sikkerhet.AuditLoggerEvent
2424
import no.nav.familie.tilbake.sikkerhet.Behandlerrolle
25-
import no.nav.familie.tilbake.sikkerhet.Rolletilgangssjekk
2625
import no.nav.familie.tilbake.sikkerhet.TilgangAdvice
2726
import no.nav.familie.tilbake.sikkerhet.TilgangService
2827
import no.nav.security.token.support.core.api.ProtectedWithClaims
@@ -56,11 +55,17 @@ class BehandlingController(
5655
consumes = [MediaType.APPLICATION_JSON_VALUE],
5756
produces = [MediaType.APPLICATION_JSON_VALUE],
5857
)
59-
@Rolletilgangssjekk(Behandlerrolle.SAKSBEHANDLER, "Oppretter tilbakekreving", AuditLoggerEvent.CREATE)
6058
fun opprettBehandling(
6159
@Valid @RequestBody
6260
opprettTilbakekrevingRequest: OpprettTilbakekrevingRequest,
6361
): Ressurs<String> {
62+
tilgangAdvice.validerTilgangFagsystemOgFagsakId(
63+
fagsystem = opprettTilbakekrevingRequest.fagsystem,
64+
eksternFagsakId = opprettTilbakekrevingRequest.eksternFagsakId,
65+
minimumBehandlerrolle = Behandlerrolle.SAKSBEHANDLER,
66+
auditLoggerEvent = AuditLoggerEvent.CREATE,
67+
handling = "Oppretter tilbakekreving",
68+
)
6469
val behandling = behandlingService.opprettBehandling(opprettTilbakekrevingRequest)
6570
return Ressurs.success(behandling.eksternBrukId.toString(), melding = "Behandling er opprettet.")
6671
}
@@ -71,11 +76,17 @@ class BehandlingController(
7176
consumes = [MediaType.APPLICATION_JSON_VALUE],
7277
produces = [MediaType.APPLICATION_JSON_VALUE],
7378
)
74-
@Rolletilgangssjekk(Behandlerrolle.SAKSBEHANDLER, "Oppretter tilbakekreving manuelt", AuditLoggerEvent.CREATE)
7579
fun opprettBehandlingManuellTask(
7680
@Valid @RequestBody
7781
opprettManueltTilbakekrevingRequest: OpprettManueltTilbakekrevingRequest,
7882
): Ressurs<String> {
83+
tilgangAdvice.validerTilgangYtelsetypeOgFagsakId(
84+
ytelsestype = opprettManueltTilbakekrevingRequest.ytelsestype,
85+
eksternFagsakId = opprettManueltTilbakekrevingRequest.eksternFagsakId,
86+
minimumBehandlerrolle = Behandlerrolle.SAKSBEHANDLER,
87+
auditLoggerEvent = AuditLoggerEvent.CREATE,
88+
handling = "Oppretter tilbakekreving manuelt",
89+
)
7990
behandlingService.opprettBehandlingManuellTask(opprettManueltTilbakekrevingRequest)
8091
return Ressurs.success("Manuell opprettelse av tilbakekreving er startet")
8192
}
@@ -86,11 +97,16 @@ class BehandlingController(
8697
consumes = [MediaType.APPLICATION_JSON_VALUE],
8798
produces = [MediaType.APPLICATION_JSON_VALUE],
8899
)
89-
@Rolletilgangssjekk(Behandlerrolle.SAKSBEHANDLER, "Oppretter tilbakekrevingsrevurdering", AuditLoggerEvent.CREATE)
90100
fun opprettRevurdering(
91101
@Valid @RequestBody
92102
opprettRevurderingDto: OpprettRevurderingDto,
93103
): Ressurs<String> {
104+
tilgangAdvice.validerTilgangBehandlingID(
105+
behandlingId = opprettRevurderingDto.originalBehandlingId,
106+
minimumBehandlerrolle = Behandlerrolle.SAKSBEHANDLER,
107+
auditLoggerEvent = AuditLoggerEvent.CREATE,
108+
handling = "Oppretter tilbakekrevingsrevurdering",
109+
)
94110
val behandling = behandlingService.opprettRevurdering(opprettRevurderingDto)
95111
return Ressurs.success(behandling.eksternBrukId.toString(), melding = "Revurdering er opprettet.")
96112
}

‎src/main/kotlin/no/nav/familie/tilbake/api/DokumentController.kt

+40-9
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ import no.nav.familie.tilbake.kontrakter.Ressurs
1717
import no.nav.familie.tilbake.kontrakter.tilbakekreving.ForhåndsvisVarselbrevRequest
1818
import no.nav.familie.tilbake.sikkerhet.AuditLoggerEvent
1919
import no.nav.familie.tilbake.sikkerhet.Behandlerrolle
20-
import no.nav.familie.tilbake.sikkerhet.Rolletilgangssjekk
2120
import no.nav.familie.tilbake.sikkerhet.TilgangAdvice
2221
import no.nav.security.token.support.core.api.ProtectedWithClaims
2322
import org.springframework.http.MediaType
@@ -42,23 +41,33 @@ class DokumentController(
4241
) {
4342
@Operation(summary = "Bestill brevsending")
4443
@PostMapping("/bestill")
45-
@Rolletilgangssjekk(Behandlerrolle.SAKSBEHANDLER, "Sender brev", AuditLoggerEvent.CREATE)
4644
fun bestillBrev(
4745
@RequestBody @Valid
4846
bestillBrevDto: BestillBrevDto,
4947
): Ressurs<Nothing?> {
48+
tilgangAdvice.validerTilgangBehandlingID(
49+
behandlingId = bestillBrevDto.behandlingId,
50+
minimumBehandlerrolle = Behandlerrolle.SAKSBEHANDLER,
51+
auditLoggerEvent = AuditLoggerEvent.CREATE,
52+
handling = "Sender brev",
53+
)
5054
val maltype: Dokumentmalstype = bestillBrevDto.brevmalkode
5155
dokumentbehandlingService.bestillBrev(bestillBrevDto.behandlingId, maltype, bestillBrevDto.fritekst)
5256
return Ressurs.success(null)
5357
}
5458

5559
@Operation(summary = "Forhåndsvis brev")
5660
@PostMapping("/forhandsvis")
57-
@Rolletilgangssjekk(Behandlerrolle.SAKSBEHANDLER, "Forhåndsviser brev", AuditLoggerEvent.ACCESS)
5861
fun forhåndsvisBrev(
5962
@RequestBody @Valid
6063
bestillBrevDto: BestillBrevDto,
6164
): Ressurs<ByteArray> {
65+
tilgangAdvice.validerTilgangBehandlingID(
66+
behandlingId = bestillBrevDto.behandlingId,
67+
minimumBehandlerrolle = Behandlerrolle.SAKSBEHANDLER,
68+
auditLoggerEvent = AuditLoggerEvent.ACCESS,
69+
handling = "Forhåndsviser brev",
70+
)
6271
val dokument: ByteArray =
6372
dokumentbehandlingService.forhåndsvisBrev(
6473
bestillBrevDto.behandlingId,
@@ -73,33 +82,55 @@ class DokumentController(
7382
"/forhandsvis-varselbrev",
7483
produces = [MediaType.APPLICATION_PDF_VALUE],
7584
)
76-
@Rolletilgangssjekk(Behandlerrolle.SAKSBEHANDLER, "Forhåndsviser brev", AuditLoggerEvent.ACCESS)
7785
fun hentForhåndsvisningVarselbrev(
7886
@Valid @RequestBody
7987
forhåndsvisVarselbrevRequest: ForhåndsvisVarselbrevRequest,
80-
): ByteArray = varselbrevService.hentForhåndsvisningVarselbrev(forhåndsvisVarselbrevRequest)
88+
): ByteArray {
89+
tilgangAdvice.validerTilgangFagsystemOgFagsakId(
90+
fagsystem = forhåndsvisVarselbrevRequest.fagsystem,
91+
eksternFagsakId = forhåndsvisVarselbrevRequest.eksternFagsakId,
92+
minimumBehandlerrolle = Behandlerrolle.SAKSBEHANDLER,
93+
auditLoggerEvent = AuditLoggerEvent.ACCESS,
94+
handling = "Forhåndsviser brev",
95+
)
96+
return varselbrevService.hentForhåndsvisningVarselbrev(forhåndsvisVarselbrevRequest)
97+
}
8198

8299
@Operation(summary = "Forhåndsvis henleggelsesbrev")
83100
@PostMapping(
84101
"/forhandsvis-henleggelsesbrev",
85102
produces = [MediaType.APPLICATION_JSON_VALUE],
86103
)
87-
@Rolletilgangssjekk(Behandlerrolle.SAKSBEHANDLER, "Forhåndsviser henleggelsesbrev", AuditLoggerEvent.ACCESS)
88104
fun hentForhåndsvisningHenleggelsesbrev(
89105
@Valid @RequestBody
90106
dto: ForhåndsvisningHenleggelsesbrevDto,
91-
): Ressurs<ByteArray> = Ressurs.success(henleggelsesbrevService.hentForhåndsvisningHenleggelsesbrev(dto.behandlingId, dto.fritekst))
107+
): Ressurs<ByteArray> {
108+
tilgangAdvice.validerTilgangBehandlingID(
109+
behandlingId = dto.behandlingId,
110+
minimumBehandlerrolle = Behandlerrolle.SAKSBEHANDLER,
111+
auditLoggerEvent = AuditLoggerEvent.ACCESS,
112+
handling = "Forhåndsviser henleggelsesbrev",
113+
)
114+
return Ressurs.success(henleggelsesbrevService.hentForhåndsvisningHenleggelsesbrev(dto.behandlingId, dto.fritekst))
115+
}
92116

93117
@Operation(summary = "Forhåndsvis vedtaksbrev")
94118
@PostMapping(
95119
"/forhandsvis-vedtaksbrev",
96120
produces = [MediaType.APPLICATION_JSON_VALUE],
97121
)
98-
@Rolletilgangssjekk(Behandlerrolle.SAKSBEHANDLER, "Forhåndsviser brev", AuditLoggerEvent.ACCESS)
99122
fun hentForhåndsvisningVedtaksbrev(
100123
@Valid @RequestBody
101124
hentForhåndsvisningVedtaksbrevRequest: HentForhåndvisningVedtaksbrevPdfDto,
102-
): Ressurs<ByteArray> = Ressurs.success(vedtaksbrevService.hentForhåndsvisningVedtaksbrevMedVedleggSomPdf(hentForhåndsvisningVedtaksbrevRequest))
125+
): Ressurs<ByteArray> {
126+
tilgangAdvice.validerTilgangBehandlingID(
127+
behandlingId = hentForhåndsvisningVedtaksbrevRequest.behandlingId,
128+
minimumBehandlerrolle = Behandlerrolle.SAKSBEHANDLER,
129+
auditLoggerEvent = AuditLoggerEvent.ACCESS,
130+
handling = "Forhåndsviser brev",
131+
)
132+
return Ressurs.success(vedtaksbrevService.hentForhåndsvisningVedtaksbrevMedVedleggSomPdf(hentForhåndsvisningVedtaksbrevRequest))
133+
}
103134

104135
@Operation(summary = "Hent vedtaksbrevtekst")
105136
@GetMapping(

‎src/main/kotlin/no/nav/familie/tilbake/api/ManuellBrevmottakerController.kt

+6-6
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import no.nav.familie.tilbake.dokumentbestilling.manuell.brevmottaker.ManuellBre
99
import no.nav.familie.tilbake.kontrakter.Ressurs
1010
import no.nav.familie.tilbake.sikkerhet.AuditLoggerEvent
1111
import no.nav.familie.tilbake.sikkerhet.Behandlerrolle
12-
import no.nav.familie.tilbake.sikkerhet.Rolletilgangssjekk
1312
import no.nav.familie.tilbake.sikkerhet.TilgangAdvice
1413
import no.nav.security.token.support.core.api.ProtectedWithClaims
1514
import org.springframework.http.MediaType
@@ -99,15 +98,16 @@ class ManuellBrevmottakerController(
9998

10099
@Operation(summary = "Fjerner manuell brevmottaker")
101100
@DeleteMapping(path = ["/{behandlingId}/{manuellBrevmottakerId}"])
102-
@Rolletilgangssjekk(
103-
minimumBehandlerrolle = Behandlerrolle.SAKSBEHANDLER,
104-
handling = "Fjerner manuell brevmottaker",
105-
auditLoggerEvent = AuditLoggerEvent.UPDATE,
106-
)
107101
fun fjernManuellBrevmottaker(
108102
@PathVariable behandlingId: UUID,
109103
@PathVariable manuellBrevmottakerId: UUID,
110104
): Ressurs<String> {
105+
tilgangAdvice.validerTilgangBehandlingID(
106+
behandlingId = behandlingId,
107+
minimumBehandlerrolle = Behandlerrolle.SAKSBEHANDLER,
108+
auditLoggerEvent = AuditLoggerEvent.UPDATE,
109+
handling = "Fjerner manuell brevmottaker",
110+
)
111111
manuellBrevmottakerService.fjernBrevmottaker(behandlingId, manuellBrevmottakerId)
112112
return Ressurs.success("", melding = "Manuell brevmottaker er fjernet")
113113
}

‎src/main/kotlin/no/nav/familie/tilbake/sikkerhet/Rolletilgangssjekk.kt

-10
This file was deleted.

0 commit comments

Comments
 (0)