Skip to content

Conversation

@cyberphysic4l
Copy link
Contributor

Description of change

The ConsensusTransactionKey for MisbehaviorReport uses authority name and the checkpoint sequence number, but this may cause ambiguity if an authority submits more than one report in one checkpoint. It is better to use the digest of the misbehavior report.

This PR adds a digest to the misbehavior reports and uses this for the consensus transactions key.

How the change has been tested

  • Basic tests (linting, compilation, formatting, unit/integration tests)
  • Patch-specific tests (correctness, functionality coverage)
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that new and existing unit tests pass locally with my changes

Release Notes

  • Protocol:
  • Nodes (Validators and Full nodes):
  • Indexer:
  • JSON-RPC:
  • GraphQL:
  • CLI:
  • Rust SDK:
  • REST API:

@vercel
Copy link

vercel bot commented Dec 4, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

6 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
apps-backend Ignored Ignored Dec 4, 2025 4:10pm
apps-ui-kit Ignored Ignored Dec 4, 2025 4:10pm
iota-evm-bridge Ignored Ignored Dec 4, 2025 4:10pm
iota-multisig-toolkit Ignored Ignored Dec 4, 2025 4:10pm
rebased-explorer Ignored Ignored Dec 4, 2025 4:10pm
wallet-dashboard Ignored Ignored Dec 4, 2025 4:10pm

@cyberphysic4l cyberphysic4l changed the title Protocol research/feat/misbehavior report keys fix(iota-types): use digest for ConsensusTransactionKey of MisbehaviorReport Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants