Skip to content

Support signature packets#22

Draft
shniubobo wants to merge 5 commits intomasterfrom
feat/types/sig
Draft

Support signature packets#22
shniubobo wants to merge 5 commits intomasterfrom
feat/types/sig

Conversation

@shniubobo
Copy link
Owner

@shniubobo shniubobo commented May 23, 2025

This has been a lot of work, and it has not been completed yet. Several Convert implementations are still buggy, or unimplemented. And Render has not been implemented for those signature types.

Todo list

  • Fix buggy Convert implementations:
    • gen_bitflags_convert_impls!
    • SignatureSubpacketData::EmbeddedSignature
  • impl Convert<pgp::packet::signature::subpacket::SubpacketLength> for SignatureSubpacketLength
    • Patch sequoia-openpgp, or wait for this issue to be resolved
  • Implement Render
  • Write tests for Display implementations in packet
  • Write missing (doc) comments

shniubobo added 2 commits May 23, 2025 04:34
..., and of course add signature-related types.

Some `Convert` implementations are still buggy, and are to be fixed in
the next several commits.

Changelog: detail
`sequoia-openpgp` does not expose the various `parse` methods, so we
have to patch it to make them public.

Ref: shniubobo/sequoia@e906c1c
Changelog: detail
shniubobo added 3 commits May 23, 2025 04:52
... to stop `wasm-opt` from complaining. The exact reason why this is
needed is not known to me.

Changelog: detail
`sequoia-openpgp` gives us the embedded signature as one big field, so
we need to retrieve field lengths within it with extra steps. Another
patch has been done to make this possible.

Before this commit, embedded signatures resulted in
`Error::SpanNotFound`, and this commit fixes that.

Ref: shniubobo/sequoia@aebe70d
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.

1 participant