Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
bjorntore opened this issue May 10, 2024 · 1 comment
Open
Assignees
Labels
kind/bug Something isn't working

Comments

@bjorntore
Copy link
Contributor

bjorntore commented May 10, 2024

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.

@bjorntore bjorntore added the kind/bug Something isn't working label May 10, 2024
@bjorntore bjorntore self-assigned this May 14, 2024
@veronikaandersen
Copy link
Contributor

Denne er litt i samme gate ser det ut som #656 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

3 participants