contentauth / c2pa-js

JavaScript SDK for displaying and validating C2PA data
https://opensource.contentauthenticity.org
MIT License
74 stars 40 forks source link

Update c2pa-rs to 0.16.1 and added fix for remote URLs #73

Closed dkozma closed 1 year ago

dkozma commented 1 year ago

Changes in this pull request

Adds validation for remote manifests that are not properly-formed URLs. The test that triggered this had a JUMBF URL in the XMP dcterms:provenance tag, but no claim data. c2patool reported no claim data - this updates our code to do the same.

Types of changes

Checklist

dkozma commented 1 year ago

@emensch I've added a "do not merge" label until I've had the opportunity to add tests to this, but wanted to get your 👀 on it.

github-actions[bot] commented 1 year ago

✅ Changeset has been found!

This will create and publish a new version to NPM after a successful merge.

dkozma commented 1 year ago

Will add tests for this particular issue once we have a fixture with proper usage rights we can add to the repo.