Skip to content

Deprecate digestSri.#1629

Open
msporny wants to merge 2 commits into
mainfrom
deprecate-digest-sri
Open

Deprecate digestSri.#1629
msporny wants to merge 2 commits into
mainfrom
deprecate-digest-sri

Conversation

@msporny

@msporny msporny commented May 10, 2026

Copy link
Copy Markdown
Member

Addresses issue #1628 by deprecating digestSri.


Preview | Diff

@longpd longpd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I agree and support this PR.

@kezike kezike left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good, thanks 👍🏾

@msporny

msporny commented May 20, 2026

Copy link
Copy Markdown
Member Author

@shigeya you noted that you plan to provide feedback on this PR. Please provide feedback soon as currently there are no objections to merging this PR.

@shigeya

shigeya commented May 20, 2026

Copy link
Copy Markdown
Contributor

@shigeya you noted that you plan to provide feedback on this PR. Please provide feedback soon as currently there are no objections to merging this PR.

I've posted my thoughts in the issue #1628.

@iherman

iherman commented May 20, 2026

Copy link
Copy Markdown
Member

This was discussed during the vcwg meeting on 20 May 2026.

View the transcript

w3c/vc-data-model#1629

<manu> w3c/vc-data-model#1628

manu: PR to deprecate digest SRI - lots of +1s but also a push back on the issue
… raised a number of things, don't want to merge right now because it feels like he is objecting, would be good for others in group to chime in
… main concern with objections is that they are not technical in nature
… we've got to do something with the PR and issue, other opinions would be helpful in the thread

Brent: we do not have consensus to merge the PR (chair hat on) - implemented against DigestSRI our changing the spec now or changing in the future is requiring implementations to change, which is a consideration we have to take seriously
… please engage on the issue
… important discussion group needs to have


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.

7 participants