Skip to content

feat: Add redaction logic#105

Merged
cdmurph32 merged 7 commits into
mainfrom
redact_pii_assertions
May 5, 2026
Merged

feat: Add redaction logic#105
cdmurph32 merged 7 commits into
mainfrom
redact_pii_assertions

Conversation

@cdmurph32
Copy link
Copy Markdown
Contributor

Import C2paReason type from contentauth/c2pa-rs#2100

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 4, 2026

🦋 Changeset detected

Latest commit: b60116e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@contentauth/c2pa-types Patch
@contentauth/c2pa-wasm Patch
@contentauth/c2pa-web Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

setThumbnailFromBlob: (format: string, blob: Blob) => Promise<void>;

/**
* Redact an assertion from an ingredient manifest.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Or a thumbnail? Does this handle redacting thumbnails too (it should, since they have/are URI, but gathering them is different than gathering action assertions).
It may be worth adding one test that redacts a thumbnail, to verify there are no surprises here.
(The challenging ones in native SDKs were to redact the thumbnail on an ingredient added as ingredient, to redact the thumbnail add_ingredient adds when the Rust feature to autogenerate thumbnails is here).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sounds like a good idea.

@tmathern tmathern requested review from ale-adobe and emensch May 4, 2026 23:22
@cdmurph32 cdmurph32 merged commit 9e03683 into main May 5, 2026
2 checks passed
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.

2 participants