Skip to content

SigId validation#55

Open
coot wants to merge 7 commits into
mainfrom
coot/sig-id
Open

SigId validation#55
coot wants to merge 7 commits into
mainfrom
coot/sig-id

Conversation

@coot
Copy link
Copy Markdown
Contributor

@coot coot commented May 13, 2026

List of changes

  • TraceLocalMsgSubmission: removed unused type parameter
  • Sig: Verbose ToJSON instance
  • Removed instance: ToJSON (AnyMessage SigSubmissionV2)
  • Sig: changed CBOR encoding
  • Sig: use Hash Blake2b_256 for sigId field
  • LogFormatting for AnyMessage (SigSubmissionV2 SigId (Sig crypto))
  • Added changelog entry

Checklist

  • related issue
  • My changes generate no new warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works

@github-project-automation github-project-automation Bot moved this to In Progress in Ouroboros Network May 13, 2026
@coot coot self-assigned this May 13, 2026
@coot coot linked an issue May 13, 2026 that may be closed by this pull request
coot added 3 commits May 13, 2026 19:00
It is not needed, since we use the new tracing system.
The encoding is changed according to:
cardano-foundation/CIPs#1185
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Signature id validation

1 participant