search

Altinn / app-lib-dotnet

Libraries used in Altinn Apps
BSD 3-Clause "New" or "Revised" License
9 stars 10 forks source link

Double signature if you use both /action and process/next with action #642

Open bjorntore opened 4 months ago

bjorntore commented 4 months ago

Description of the bug

Det oppstår dobbelt signatur.json-objekt hvis man utfører sign via actions-endepunktet og deretter via process/next med sign som action, selv ved max-count: 1 på datatypen. Ved eventuelle flere steg vil da validering feile, ettersom det eksisterer for mange signaturobjekter i forhold til max-count. Man kan heller ikke gjøre signering via action-endepunktet og deretter kjøre process/next uten action. Det feiler.

DIBK ønsker gjerne at siste signatur overskriver eventuell tidligere utført via action-endepunktet. Konsekvenser av dette bør diskuteres først. Alternativet er feilmelding om man prøver å signere to ganger på identiske data? Process/next uten action bør fungere dersom man alt har signert via actions-endepunkt.

Steps To Reproduce

  1. Ha en app med grunnleggende signering satt opp.
  2. Sørg for at det finnes et dataelement av datatypen(e) som skal signeres, slik at vi har noe å signere på.
  3. Signer via /action-endepunktet, feks. i Postman.
  4. Prøv å kjør process/next uten sign-action. Dette tror vi gir feilmelding nå. Burde sikkert ikke det.
  5. Prøv å kjør process/next med sign-action. Da bør gå igjennom, men du ender opp med to signeringobjekter, som vil feile i validering i eventuelle senere steg.

Additional Information

Se Slack-samtale om issuet: https://altinn.slack.com/archives/CCQEQKGJD/p1714044845070929.

veronikaandersen commented 3 months ago

Denne er litt i samme gate ser det ut som https://github.com/Altinn/app-lib-dotnet/issues/656 :)