Altinn / Altinn/altinn-platform-validation-tests

Ventende endringsforespørsler overlever systembrukeren og systemet

Open
#466 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

k6
Dominant language
JavaScript
Stars
2
Forks
1
Avg merge
1d 3h
Merged PRs (30d)
124

Description

Ventende endringsforespørsler overlever både systembrukeren de gjelder og systemet de hører til, og fortsetter å bli listet ut for et system som ikke finnes lenger.

Målt i at23, med to endringsforespørsler med status New på en systembruker vi selv arrangerte:

pending before cleanup: 2, statuses ["New","New"]
delete system user with pending change requests: true
delete system with pending change requests: 200
systems left in the register: 0
change requests listed after the system is gone: 2

Begge slettene svarer greit, systemet er borte fra GET /systemregister/vendor, og GET /systemuser/changerequest/vendor/bysystem/{systemId} returnerer likevel de to forespørslene etterpå. De kan ikke lenger godkjennes av noen, siden systembrukeren de peker på er slettet, og de kan ikke ryddes av noe annet enn en eksplisitt tilbaketrekking mot en id noen har tatt vare på.

Det som gjør dette til noe mer enn et skjønnhetsproblem er at det akkumulerer. Hver testkjøring som stopper mellom opprettelse og tilbaketrekking legger igjen sine, og de blir liggende for godt i alle miljøer.

Til sammenligning: en vanlig systembrukerforespørsel med status New kan trekkes tilbake på samme måte, men den hører til systemet og ikke til systembrukeren, så den forsvinner sammen med systemet.

Spørsmål til authentication-teamet: skal endringsforespørsler ryddes når systembrukeren eller systemet slettes, eller er det meningen at de blir liggende? Om de skal bli liggende, hvordan er de tenkt ryddet?

Testsiden er dekket i mellomtiden: sweepPendingChangeRequests i #459 trekker tilbake det som er ventende før systembrukeren slettes.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the lifecycle in at23 using the system-user and system deletion calls, then inspect the change-request listing endpoint and the sweepPendingChangeRequests workaround from #459. The authentication team must first decide whether deletion should remove pending requests or define another cleanup path; completion requires that the chosen behavior prevents orphaned requests from remaining listed.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
authentication
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
32/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.