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

update to idempotent transaction operations #4

Open
3 tasks
ryankurte opened this issue Feb 13, 2023 · 0 comments
Open
3 tasks

update to idempotent transaction operations #4

ryankurte opened this issue Feb 13, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@ryankurte
Copy link
Contributor

ryankurte commented Feb 13, 2023

with the existing transaction computation applying the same event twice will alter the digest and actual computation, this is likely to cause problems in the event of packet loss, though whether this is a problem remains to be seen when bluetooth / nanox support is available.

to mitigate this we should move to idempotent operations, so that an event sent twice will only be applied once, but return the same value in both cases.

  • cache the last digest and only update on new events
  • ignore repeated state-change events if we're already in the target state
  • use indicies in APDUs to avoid applying events twice to the summary or ring signature computations
@ryankurte ryankurte added the enhancement New feature or request label Feb 13, 2023
ryankurte pushed a commit that referenced this issue Aug 2, 2024
update to v0.16.0 (fixes per functional review)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant