Skip to content

fix: harden tempo transaction verification#144

Open
emmajam wants to merge 5 commits into
mainfrom
ejh/pympp-tx-verification-hardening
Open

fix: harden tempo transaction verification#144
emmajam wants to merge 5 commits into
mainfrom
ejh/pympp-tx-verification-hardening

Conversation

@emmajam
Copy link
Copy Markdown
Member

@emmajam emmajam commented Jun 1, 2026

Summary

  • Enforce challenge-bound attribution memo checks for Tempo transaction credentials, matching the hash credential path.
  • Reserve computed raw transaction hashes before broadcast when replay protection storage is configured.
  • On duplicate transaction credentials, fetch and validate the existing receipt instead of rebroadcasting.
  • Release reservations for submission errors while keeping receipt/validation failures reserved for retry safety.
  • Added a patch changelog entry and regression tests. Confirmed uv run ruff check ., uv run pyright, and uv run pytest -q pass.

Comment thread tests/test_tempo.py
Comment thread src/mpp/methods/tempo/intents.py Outdated
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.

2 participants