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

RFC001 - Clarification whether PAR si required or not #116

Open
jtalir opened this issue Nov 29, 2024 · 4 comments
Open

RFC001 - Clarification whether PAR si required or not #116

jtalir opened this issue Nov 29, 2024 · 4 comments

Comments

@jtalir
Copy link
Contributor

jtalir commented Nov 29, 2024

Based on our current experiences, it is not easy to write interoperable issuer according to RFC001. There are a lot of things that are optional in OID4VCI but underspecified in RFC001. One of these things is Pushed Authorization Request. Example of authorization metadata shows it as required, but example od Authorization request is not compliant with how PAR request should look like.

If PAR is required, it should be clearly states it RFC and examples should be update accordingly.

If PAR is not required, than we don't have interoperability since at least Lissi wallet requires PAR so any issuer that will not implement PAR will not be compliant with at least one compliant wallet.

@jtalir
Copy link
Contributor Author

jtalir commented Nov 29, 2024

RFC ITB issuer requires PAR https://dss.aegean.gr/rfc-issuer/.well-known/oauth-authorization-server so all RFC compliant wallet should be able to handle it. So let's make it mandatory and update authorization request accordingly.

@endimion
Copy link
Collaborator

Actually, the ITB supports PAR but doesnt mandate it...but overall I agree we need to reduce a bit the options to ensure ineroperabilty for this pilot phase...

@jtalir
Copy link
Contributor Author

jtalir commented Nov 29, 2024

Actually, the ITB supports PAR but doesnt mandate it...but overall I agree we need to reduce a bit the options to ensure ineroperabilty for this pilot phase...

This is contrary to what metadata says "require_pushed_authorization_request": "true" :-). And it makes it even more challenging since we don't know if all wallets support PAR or not.

@endimion
Copy link
Collaborator

oops had forgotten that.. I will change to false. But internally the process continues even if the wallet doesnt support PAR. This has to do with a previous issue I raised if we will make HAIP mandatory or not. But @andreasabr had some concerns about it....

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

2 participants