OpenIDVPs: register vp_token response type

Issue #1557 resolved
Torsten Lodderstedt created an issue

No description provided.

Comments (9)

  1. Thomas Bellebaum

    Since OpenID4VP is an “addon” to OAuth and OIDC, we probably also want the following:

    • code vp_token
    • id_token vp_token
    • token vp_token
    • code id_token vp_token
    • code token vp_token
    • id_token token vp_token
    • code id_token token vp_token

    Why was this scheme chosen again? The registry currently is listing any combination of code, id_token and token anyway…

  2. Torsten Lodderstedt reporter

    The verifier requests credential presentation via the presentation_definition request parameter.

    The new response type „vp_token“ is only needed to fulfill OAuth protocol requirements for the case where the vp token is the only artifact the AS responds. One just needs to specify something.

    In all other cases, there is a pre-existing response type, like „id_token“ or „code“ and the vp token is returned with the id token or the access token. So no need to register other response types and require verifiers to use them, it would just duplicate information.

    Note: retuning access tokens in the front channel is discouraged by the OAuth Security BCP. Another reason not to do it.

  3. Thomas Bellebaum

    Take id_token vp_token as an example (a use case would be OpenID4VP over SIOP).
    This expects to be returned two tokens. Also, there are probably request parameters scope and credential_definition.

    By the logic above, we do not need to include vp_token in the response type, since we have a parameter called credential_definition.
    But by the same logic, we could instead also omit id_token, since we check for openid in the scope parameter.

    Applying the same logic to both seems more consistent.

  4. Kristina Yasuda

    +1 to Mike’s comment. we should not enable a whole list of combinations other than vp_token and vp_token id_token and only those two should be registered.

  5. Kristina Yasuda

    there is an IANA registry for vp_token. let's open a separate issue once we are closer to final and are ready to actually register with IANA

  6. Log in to comment