Skip to content

Update naming and parameters of authorization requests#848

Merged
jkoenig134 merged 5 commits intorelease/openid4vcfrom
update-naming-and-parameters-of-authorization-requests
Nov 17, 2025
Merged

Update naming and parameters of authorization requests#848
jkoenig134 merged 5 commits intorelease/openid4vcfrom
update-naming-and-parameters-of-authorization-requests

Conversation

@jkoenig134
Copy link
Contributor

@jkoenig134 jkoenig134 commented Nov 17, 2025

Readiness checklist

  • I added/updated tests.
  • I ensured that the PR title is good enough for the changelog.
  • I labeled the PR.
  • I self-reviewed the PR.

Description

This PR updates multiple issues the app has with the current openid4vc behavior:

  1. (most important) it is very hard to detect which credentials were used to resolve the authorization request. I resolved this with adding the property usedCredentials to the response which shows the requests that will be used to accept the request with.
  2. naming. Everything regarding presentations is named quite strange - I tried to fix some of the issues.

@jkoenig134 jkoenig134 added the enhancement New feature or request label Nov 17, 2025
@jkoenig134 jkoenig134 requested a review from tnotheis November 17, 2025 11:23
@jkoenig134
Copy link
Contributor Author

@Milena-Czierlinski FYI.

@jkoenig134 jkoenig134 enabled auto-merge (squash) November 17, 2025 11:30
@jkoenig134 jkoenig134 disabled auto-merge November 17, 2025 11:34
@jkoenig134 jkoenig134 merged commit 16f10fe into release/openid4vc Nov 17, 2025
15 of 16 checks passed
@jkoenig134 jkoenig134 deleted the update-naming-and-parameters-of-authorization-requests branch November 17, 2025 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants