c2pa-rs icon indicating copy to clipboard operation
c2pa-rs copied to clipboard

Expose `SignatureInfo` publicly

Open ok-nick opened this issue 1 year ago • 2 comments

Changes in this pull request

Expose SignatureInfo publicly (already used in public API).

Checklist

  • [x] This PR represents a single feature, fix, or change.
  • [x] All applicable changes have been documented.
  • [x] Any TO DO items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.

ok-nick avatar Jul 10 '24 17:07 ok-nick

I think if we are making the struct public, it makes sense to also make the fields public. We could alternatively provide getters.

gpeacock avatar Jul 10 '24 18:07 gpeacock

If user's are deserializing into the struct, I think it makes sense to have the fields public so they know what they're deserializing into.

ok-nick avatar Jul 10 '24 18:07 ok-nick