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

feat: Add get_provenance_cert_chain to Reader #580

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

iameli
Copy link
Contributor

@iameli iameli commented Sep 11, 2024

I found myself needing to inspect the cert chain present in a C2PA asset over in c2pa-go. This reader interface seemed to be the only piece exposed in the UniFFI bindings, so I added a helper function here. I have no idea if I'm doing it the right way, but it does work.

@gpeacock
Copy link
Collaborator

I'm trying to keep the API space to a minimum, so let me think about this. I'm planning to expose the detailed manifest store report to this API (with something like json-detailed() and that could include the certs from every manifest in the store.

@scouten-adobe scouten-adobe changed the title reader: add get_provenance_cert_chain feat: Add get_provenance_cert_chain to Reader Oct 7, 2024
@scouten-adobe
Copy link
Collaborator

@iameli would you please enable maintainers' contributions? This would allow us to update to base branch. Otherwise, we'll keep asking you to update and hope that we notice at the right time. 😐

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.

3 participants