Skip to content

Conversation

@Ryanmtate
Copy link
Contributor

@Ryanmtate Ryanmtate commented Jan 22, 2026

Description

This pull request enhances the Mdoc credential handling in the Rust codebase, focusing on improved interoperability and testability. The main changes introduce a new constructor for creating Mdoc instances from CBOR-encoded issuer-signed data and add a comprehensive test to ensure correct CBOR parsing and object construction. Additionally, the dependency on the isomdl crate is updated for compatibility and feature improvements.

New functionality and test coverage:

  • Added a new Mdoc constructor, new_from_cbor_encoded_issuer_signed, allowing direct creation of an Mdoc from CBOR-encoded issuer-signed bytes, improving interoperability with external data sources.
  • Introduced a unit test that decodes a base64-encoded CBOR payload, parses it into an IssuerSigned object, and constructs an Mdoc, verifying the new constructor and CBOR parsing logic.

Dependency updates:

  • Updated the isomdl crate dependency to a newer commit (7959c8b), ensuring compatibility with the latest features and bug fixes.

@Ryanmtate Ryanmtate requested a review from sbihel January 22, 2026 22:37
@Ryanmtate Ryanmtate force-pushed the feat/cbor-bytes-issuer-signed-mdoc-constructor branch from ae07c61 to d9d0cfa Compare January 22, 2026 22:37
@Ryanmtate
Copy link
Contributor Author

depends on spruceid/isomdl#117

@Ryanmtate Ryanmtate force-pushed the feat/cbor-bytes-issuer-signed-mdoc-constructor branch from d9d0cfa to efda999 Compare January 22, 2026 22:39
@Ryanmtate Ryanmtate changed the title use isomdl 7959c8b to handle digest id mapping, add mdoc constructor … Add mdoc constructor for IssuerSignedDehydrated and NameSpacedData data Jan 27, 2026
Co-authored-by: Simon Bihel <[email protected]>
@Ryanmtate Ryanmtate merged commit b33dc0a into main Jan 29, 2026
4 checks passed
@Ryanmtate Ryanmtate deleted the feat/cbor-bytes-issuer-signed-mdoc-constructor branch January 29, 2026 20:34
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.

3 participants