Skip to content

Conversation

crandmck
Copy link
Collaborator

@crandmck crandmck commented Feb 4, 2025

Changes in this pull request

Add suggested sanity check for certs to be submitted for trust list.

Deploy preview: https://deploy-preview-188--cai-open-source.netlify.app/docs/verify-known-cert-list#checking-your-certificate .

Questions

To avoid errors, I had to add --args to the jq command here:

c2patool ./image.jpg trust --allowed_list ./cert.pem \
| jq --args '.manifests[].signature_info.issuer' "XYZ Inc."

Just want to confirm that's OK.

Also, you need all of these tools, which we should probably state as prerequisites to follow that section, right?

  • jq I don't think this is installed by default (on macOS anyway). Maybe use brew install jq ?
  • openssl Anyone working with certs probably has it, but for completeness...?
  • C2PA Tool. Again, someone who's gotten this far probably has it already, but still.

Closes #187

Copy link

github-actions bot commented Feb 4, 2025

@github-actions github-actions bot temporarily deployed to pull request February 4, 2025 23:30 Inactive
@crandmck crandmck requested a review from andrewhalle February 4, 2025 23:44
@github-actions github-actions bot temporarily deployed to pull request February 4, 2025 23:46 Inactive
@crandmck crandmck changed the title Add sanity check instructions for trust list certs Add docs for preliminary check for trust list certs Feb 5, 2025
@github-actions github-actions bot temporarily deployed to pull request February 5, 2025 21:55 Inactive
@crandmck crandmck requested a review from andrewhalle February 5, 2025 22:13
@github-actions github-actions bot temporarily deployed to pull request February 5, 2025 22:15 Inactive
@github-actions github-actions bot temporarily deployed to pull request February 5, 2025 22:52 Inactive
@crandmck crandmck merged commit bb041aa into main Feb 5, 2025
3 checks passed
@crandmck crandmck deleted the cert-sanity-check branch February 5, 2025 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document commands to sanity-check certs before submitting to trust list
2 participants