Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

C2PA-383: Merge in main #45

Merged
merged 26 commits into from
Mar 13, 2024

Conversation

vancejc-mt
Copy link

@vancejc-mt vancejc-mt commented Mar 11, 2024

Changes in this pull request

This PR catches up our monotype/fontSupport branch with recent changes to c2pa-rs's main branch. Necessary as current github actions are failing.

Pulled in extra changes from last night which updated the rasn-* crate dependencies, was the cause of our failures yesterday.

Checklist

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

gpeacock and others added 12 commits March 7, 2024 20:01
this gives a client a way to view the thumbnail, while still reusing the embedded one.
* Push to keep things moving

* Add ManifestStoreOptions to async bytes functions
ManifestStore::from_bytes_async()
ManifestStore::from_manifest_and_asset_bytes_async
Also some fmt/clippy fixes
Builds but unit tests do not run yet

* WASM build

* WASM fix

* Expose ManifestStoreOptions

* Fix missing anchors

* Fix misspelled var

* remove check that may be wrong

* remove issuer check

* switch to x509_parser because x509-certificate has issues

* Add support DER encoded EC signatures when validating trust lists

* Update Adobe trust root

* Small fixes for Wasm building (contentauth#354)

* Remove trust anchors from build, only leave the test certs

* Add ManifestStore::from_file_with_options

* Minor fixes

* Trust fixes

* Wasm compiler fixes

* Restored more tests

* EC sig fix

* Add ARW & NEF support

* Fix for wrong sig algoritm

* Support for allowed list

* Add Wasm compilation fixes for trust_support branch (contentauth#401)

* Fix wasm trust handler

* Fix reading lines inside PEM certificate blocks

* Wire in settings support
Remove feature flags covered by settings

* Build fixes
simplified check_trust

* Build fixes

* Test fixes

* Build fix

* another try to fix build issue

* Build fixes
Cleanup

* Fix preflight check

* Remove wrong cert from test anchors bundle

* Remove trust feature flag

* simplify manifest_store changes

* revert unintended change

* Remove ManifestStore changes
Update integration example to use loading settings from JSON

* Change feature flag setting

* Change more feature flags

* enhance trust pass through to support configuration of allowed_list and EKU config
Check signing cert before signing so that it gets checked independent of whether we validate after signing

* A little code cleanup

* Make sure ED25519 tests are disabled for WASM

* Remove Ed25519 from trust list until we can get it working in WASM

* Enable Ed25519 support

* Added debug logging for ed25519

* wasm build fix

* fix broken check

* Fixes for wasm

* Fix build failure

* Fix comment

* Code cleanup

* OCSP fix

---------

Co-authored-by: Gavin Peacock <[email protected]>
Co-authored-by: Dave Kozma <[email protected]>
…tauth#421)

Fix build error when building from crates.io with openssl_sign feature
@vancejc-mt vancejc-mt added the upstream Catch up to the upstream changes label Mar 11, 2024
@vancejc-mt vancejc-mt self-assigned this Mar 11, 2024
gpeacock and others added 2 commits March 12, 2024 09:21
* Adds ComposableManifest support to c2pa_io

* Adds Manifest::composed_manifest()
Provides a way to pre-format a manifest store for injection into a given file format.

* Remove self from Store::get_composed_manifest
@vancejc-mt vancejc-mt marked this pull request as draft March 12, 2024 21:26
@vancejc-mt
Copy link
Author

vancejc-mt commented Mar 12, 2024

Converting to draft until we see what's going on with c2pa-rs and their reliance on rasn 0.12.6 (which was just yanked earlier)

@vancejc-mt vancejc-mt marked this pull request as ready for review March 13, 2024 14:51
adds Send trait to TrustHandlerConfig

Co-authored-by: Dylan Ross <[email protected]>
Copy link

@brogdonm brogdonm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pick up the latest 0.31.0 commit merge. That way we are on a commit that is "tagged".

@vancejc-mt
Copy link
Author

Pick up the latest 0.31.0 commit merge. That way we are on a commit that is "tagged".

Good call - merged in

@vancejc-mt vancejc-mt merged commit fc39cad into monotype/fontSupport Mar 13, 2024
21 checks passed
@vancejc-mt vancejc-mt deleted the feature/C2PA-383/mergeInMain branch March 13, 2024 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upstream Catch up to the upstream changes
Projects
None yet
6 participants