openid / OpenID4VP

57 stars 20 forks source link

Allow both presentation definition and dcql query during transition period? #341

Closed TimoGlastra closed 1 day ago

TimoGlastra commented 1 day ago

During the transition period it might be beneficial to allow for both the presentation definition and dcql query to be present in an authorization request. There's no easy way for the verifier to know if the wallet already supports DCQL, and our initial design was that we include both for now, but this doesn't seem allowed:

Exactly one of the following parameters MUST be present in the Authorization Request: dcql_query, presentation_definition, presentation_definition_uri, or a scope value representing a Presentation Definition.

Then the wallet can choose, and we can avoid this problem of knowing which format to use before having interacted with a wallet. Is there a reason for the spec to be this strict? Does it hinder wallet implementations if both can be present?

The logic could be:

Sakurann commented 1 day ago

there is already #211. can you please comment there?