contentauth / c2pa-rs

Rust SDK for the core C2PA (Coalition for Content Provenance and Authenticity) specification
Other
128 stars 57 forks source link

feat: Factor status tracking infrastructure into its own crate #665

Closed scouten-adobe closed 2 weeks ago

scouten-adobe commented 2 weeks ago

Changes in this pull request

This PR (when completed) will move most of the logging and status tracking infrastructure currently in status_tracker.rs into its own crate, which can then be used in the forthcoming c2pa-crypto crate.

Checklist

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 62.38698% with 208 lines in your changes missing coverage. Please review.

Project coverage is 81.62%. Comparing base (4a0cde6) to head (8ac83ab). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
sdk/src/cose_validator.rs 26.85% 79 Missing :warning:
sdk/src/store.rs 63.15% 63 Missing :warning:
sdk/src/claim.rs 56.16% 32 Missing :warning:
sdk/src/ocsp_utils.rs 41.66% 14 Missing :warning:
sdk/src/ingredient.rs 36.84% 12 Missing :warning:
sdk/src/manifest_store.rs 70.00% 3 Missing :warning:
sdk/src/manifest_store_report.rs 75.00% 2 Missing :warning:
sdk/src/reader.rs 0.00% 2 Missing :warning:
sdk/src/openssl/rsa_signer.rs 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #665 +/- ## ========================================== + Coverage 81.41% 81.62% +0.21% ========================================== Files 94 97 +3 Lines 30835 30705 -130 ========================================== - Hits 25105 25064 -41 + Misses 5730 5641 -89 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

scouten-adobe commented 2 weeks ago

FYI most of the alerts about un-covered lines were previously un-covered. Just got flagged because I changed them to use the new APIs.