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

Reconsider PoP mechanism to get a new status attestation #32

Open
paulbastian opened this issue Apr 11, 2024 · 5 comments
Open

Reconsider PoP mechanism to get a new status attestation #32

paulbastian opened this issue Apr 11, 2024 · 5 comments
Assignees

Comments

@paulbastian
Copy link

PoP of Credential cnf key hould not be used, instead an api-key exchanged between wallet and Issuer.

Advantages are:

  • enables claim-based bound credentials
  • does not require user interaction as the PoP key may be bound to PIN/Biometrics but the api-key does not, as it is provided by the Issuer during issuance
@peppelinux
Copy link
Owner

I see the benefits @paulbastian and I believe that these should be further discussed, here some additional considerations:

  1. how it would enable claim-based bound credentials?
  2. this depends by impl. We connsider that the attestations are fetched when the user activate/execute the wallet, therefore asap after the local authentication

@OR13
Copy link
Collaborator

OR13 commented Apr 30, 2024

perhaps we say "issuer MUST authenticate the wallet", and MUST support PoP and MAY support other mechanisms, and briefly describe them?

@paulbastian
Copy link
Author

As discussed today, Oliver proposed to use a dpop key to get this at a /status endpoint. That would enable to use Referenced Token that don't have cnf claim

@peppelinux
Copy link
Owner

peppelinux commented Jun 12, 2024

@paulbastian we can do this and it is resonable, we can protect the endpoint also with DPoP using a key specialized for this purpose,

thank you

@peppelinux peppelinux self-assigned this Jun 12, 2024
@paulbastian
Copy link
Author

As we have integrated this PR into OpenID4VCI, you may easily share tokens/api-keys in the issuance to request new status assertions, instead of relying on the PoP mechanism. I would advise to move away from the PoP key in the Credential and use an Credential Format independent mechanism instead

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

No branches or pull requests

3 participants