-
Notifications
You must be signed in to change notification settings - Fork 7
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
OpenID4VP: Replace redirect_uri with response_uri for Response Mode direct_post #71
Comments
I think you are correct. |
Stumbled over this today as well - is my understanding correct that this is necessary to prevent the session fixation attack described in 12.2. If so should we add this as a recommended/required step in this profile (mandating a redirect_uri as a response to the direct_post flow)? |
If we mandate redirect, we mandate the same device flow, correct? |
OID4VP currently does not limit the usage of redirects after direct_post to only same-device flows. even though the security benefit that this redirect gives for the same-device does not apply for cross-device flows. before we make any clarifications on this topic in HAIP, we should probably make it clearer in oid4vp itself. here is the issue :) |
The HAIP specifies in Section 5 for OpenID4VP that:
However,
redirect_uri
must not be present when the Response Mode isdirect_post
as described in the OpenID4VP specs in Section 6.2-7.2:Therefore, I assume that the HAIP should use
response_uri
instead ofredirect_uri
which would align with the OpenID4VP specs.The text was updated successfully, but these errors were encountered: