Altinn / Altinn/app-lib-dotnet

Håndhev regel om ingen datamanipulering i signeringsteget

Open
#1,716 0 comments 1 reaction 0 assignees View on GitHub
kind/feature-request squad/data status/for-consideration
Dominant language
C#
Stars
8
Forks
27
Avg merge
1h 21m
Merged PRs (30d)
7

Description

### Description

Slik Altinn Studio fungerer i dag, går POST requests og DELETE requests bare på tilgangene til brukeren (read/write/delete). Også i signeringsteget. Det gjør at hvis man har skrivetilgang kan man legge til flere dataelementer (underskjema, vedlegg, hovedskjema), og har man slettetilgang kan man slette dataelementer.

Altinn innboks krever tilgangen til å slette for "Hele appen" for at brukeren skal få lov til å slette en innsending fra innboksen sin. Det betyr, at for at vi skal levere mulighet for å ikke fylle opp innboksen til brukere, må vi tillate sletting på hele appen.

Dette gjør at DELETE kall fungerer, og vi får innsendinger uten underskjema/vedlegg selv om vi har valideringer på det i utfyllingsteget, fordi de sletter ting i signeringsteget.

For å komme rundt dette, må vi ENTEN:

- Ikke trenge slettetilgang på hele appen for å slette innsendinger i innboksen
- Ha en sterk regel på at man ikke kan redigere data i signeringsteget, uavhengig av brukerrettigheter

### Additional Information

_No response_

Contributor guide

Open the contributing guide

Research direction

Begin by resolving which of the two proposed approaches is intended: changing inbox deletion permissions or blocking data edits during signing. Then trace the POST and DELETE handling for the signing step described in the issue. Done means the chosen behavior is enforced without allowing submissions to lose required subforms, attachments, or main-form data.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
api, authorization, backend
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.