Skip to content

Fix submit transport retry idempotency - #89

Merged
N-thnI merged 1 commit into
Vero-protocol:mainfrom
telemarkdigital-publisher:fix/issue-86-submit-idempotency
Aug 29, 2026
Merged

Fix submit transport retry idempotency#89
N-thnI merged 1 commit into
Vero-protocol:mainfrom
telemarkdigital-publisher:fix/issue-86-submit-idempotency

Conversation

@telemarkdigital-publisher

Copy link
Copy Markdown
Contributor

Summary

  • prepare one signed submit payload and reuse it for transport retries so timeout/all-endpoint retry paths do not reserve a fresh sequence
  • send a stable per-submit idempotency key in the request body and Idempotency-Key header
  • poll the submission status endpoint after indeterminate transport failures before retrying, while preserving BadSequence refresh/re-sign behavior

Closes #86

Verification

  • npm test -- --runInBand src/tests/contract.test.ts
  • npm test -- --runInBand
  • npm run typecheck
  • npm run lint (passes with existing warnings in src/account/tests/balances.test.ts)
  • npm run build

@N-thnI
N-thnI merged commit 0b7b23c into Vero-protocol:main Aug 29, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stop submit() from re-signing with a new sequence on transport retry

2 participants