c2pa-rs
c2pa-rs copied to clipboard
(MINOR) Initial V2 API work
This adds a new Builder and Reader model based entirely on streams. The Builder is based on the Manifest API and the Reader on ManifestStore. The builder is used to create new manifests to be signed. The reader is always read only. Ingredients are added directly to the builder. Resources such as thumbnails and icons can be accessed via Uris in resource references. Resources in a reader are always accessed via jumbf uris. The state of the builder can be captured into a zip stream and reconstructed at any time. A new CallBackSigner interface has been added which ensures the client code does not need access to private keys.
The new APIs are intended to replace the older ones. The older APIs will eventually be deprecated. The Builder and Reader APIs will be supported in language bindings for C/C++, Python and others.
Looks OK to merge behind the flags. The only things I would double check is which functions are being exposed. There are some thing I don't think we want to support.
Codecov Report
Attention: Patch coverage is 68.72749%
with 521 lines
in your changes are missing coverage. Please review.
Project coverage is 79.62%. Comparing base (
b5ce33f
) to head (5930e51
).
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@ Coverage Diff @@
## main #437 +/- ##
==========================================
- Coverage 80.31% 79.62% -0.70%
==========================================
Files 82 87 +5
Lines 25867 27276 +1409
==========================================
+ Hits 20776 21719 +943
- Misses 5091 5557 +466
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Forcing merge after all reviews passed due to MINOR VERSION check being stuck.