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

Don't require CM "mediation" to be specified explicitly (always 'required' for DC). #177

Open
samuelgoto opened this issue Oct 7, 2024 · 5 comments
Assignees

Comments

@samuelgoto
Copy link
Collaborator

Given #175, we are going to need to update every example and deployment to use mediation="required".

A few examples are, since this is backwards incompatible change:

  1. https://openid.github.io/OpenID4VP/openid-4-verifiable-presentations-wg-draft.html#name-request
  2. https://digitalcredentials.dev/
  3. https://developer.chrome.com/blog/digital-credentials-api-origin-trial

And every demo app that was built against our API.

@timcappalli
Copy link
Member

timcappalli commented Oct 7, 2024

digitalcredentials.dev is updated (digitalcredentialsdev/digitalcredentials.dev@35b915b)

@marcoscaceres
Copy link
Collaborator

Blocked on w3c/webappsec-credential-management#256

@timcappalli, we should add "Update digitalcredentials.dev" to the PR Template, so we always remember to do that.

@timcappalli
Copy link
Member

@marcoscaceres fyi: 60a44bf

@timcappalli
Copy link
Member

@samuelgoto can this be closed?

@RByers
Copy link
Member

RByers commented Dec 2, 2024

Discussed in the call today. Our implementation today doesn't require developers to specify a mediation, which is how we want to keep it. So updating the title to reflect that we need to be sure this is covered by the specs and tests. It sounds like this is mostly w3c/webappsec-credential-management#256, though I assume we might also need some text in our spec too once that's resolved, right?

@RByers RByers changed the title Update every example to include mediation="required" Don't require CM "mediation" to be specified explicitly (always 'required' for DC). Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants