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

Referanse på transmisjoner? #2052

Open
prange opened this issue Mar 14, 2025 · 1 comment
Open

Referanse på transmisjoner? #2052

prange opened this issue Mar 14, 2025 · 1 comment
Labels
needs consideration Requires additional consideration

Comments

@prange
Copy link

prange commented Mar 14, 2025

Introduction

Kunne man undersøkt om et referanse-felt på transmisjoner ville vært en god idé?

Description

I en lengre dialog der flere system er involvert er det en viss risiko for at man av ulike årsaker forsøker å gjøre en gjøre en dobbeltpostering av en transmisjon. Et referansefelt gir en klient mulighet til å sjekke om en transmisjon allerede er opprettet basert på en businessid.

Implementation

Det er relativt enkelt å legge inn et valgfritt felt som f.eks. kan hete "reference" eller "externalId" med typen String?

@MagnusSandgren
Copy link
Collaborator

Tenker du at dette feltet skal brukes client side i kombinasjon med if-match header? Eller tenker du at det skal være en idempotentKey som verifiseres av dialogporten og at man skal få feilmelding som en klient dersom samme key blir forsøkt brukt flere ganger?

Det er mulig i dag å sette transmission.id eksplisitt av klienten. Men iden må være på uuidv7. Her går det an å lage en deterministisk uuidv7 basert på en DateTime og denne "reference" eller "externalId" som du nevner. Fremgangsmåten er å generere en uuidv5 basert på referansen og omgjøre den til en v7 basert på DateTime. Er dette noe som kunne løst problemstillingen?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs consideration Requires additional consideration
Projects
Status: New issues
Development

No branches or pull requests

3 participants