[OID4VCI] Explicit client association for pre-authorized offers#50897
Open
tdiesler wants to merge 3 commits into
Open
[OID4VCI] Explicit client association for pre-authorized offers#50897tdiesler wants to merge 3 commits into
tdiesler wants to merge 3 commits into
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Decouples pre-authorized credential offers from the creating client by supporting explicit or uniquely discovered target clients.
Changes:
- Adds
target_clientrequest support and client discovery. - Updates pre-authorized issuance flows and tests.
- Adds offer-state inspection utilities.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
CredentialOfferUriRequest.java |
Adds the target-client query parameter. |
OID4VCredentialOfferPreAuthTest.java |
Tests target-client validation and discovery. |
OID4VCPublicClientPreAuthTest.java |
Specifies the public target client. |
OID4VCJWTIssuerEndpointPreAuthTest.java |
Supplies target clients in endpoint tests. |
OID4VCAuthorizationDetailsFlowPreAuthTestBase.java |
Associates authorization-details offers with a client. |
JwtPreAuthCodeHandlerTest.java |
Associates test offers with their client. |
OID4VCTestContext.java |
Exposes the context client ID. |
OID4VCredentialOfferAuthCodeTest.java |
Verifies auth-code offers remain unbound. |
OID4VCCredentialOfferCorsTest.java |
Updates target parameter names and requests. |
OID4VCSdJwtIssuingEndpointTest.java |
Supplies the SD-JWT target client. |
CredentialOfferStateUtils.java |
Adds server-side offer-state inspection. |
VerifiableCredentialOfferAction.java |
Adapts provider argument ordering. |
OID4VCIssuerEndpoint.java |
Accepts and forwards explicit target clients. |
DefaultCredentialOfferProvider.java |
Implements target-client discovery and validation. |
CredentialOfferProvider.java |
Updates provider parameter ordering. |
tdiesler
force-pushed
the
ghi50578-preauth-client
branch
from
July 14, 2026 19:04
bb30691 to
b99783b
Compare
…d for offer creation and redemption closes keycloak#48188 Signed-off-by: Thomas Diesler <tdiesler@proton.me> * Stop binding authorization-code credential offers to the creating client (targetClientId = null for auth-code offers). * Keep pre-authorized code offers bound to the authenticated client. * Update Javadoc section headings to explicitly name the corresponding grant types.
tdiesler
force-pushed
the
ghi50578-preauth-client
branch
from
July 15, 2026 06:55
b99783b to
baa1a5a
Compare
tdiesler
marked this pull request as ready for review
July 15, 2026 06:56
tdiesler
marked this pull request as draft
July 15, 2026 16:11
tdiesler
force-pushed
the
ghi50578-preauth-client
branch
4 times, most recently
from
July 16, 2026 10:03
179ef83 to
54a9776
Compare
tdiesler
marked this pull request as ready for review
July 16, 2026 12:18
tdiesler
force-pushed
the
ghi50578-preauth-client
branch
from
July 16, 2026 14:15
54a9776 to
c9c73c8
Compare
tdiesler
force-pushed
the
ghi50578-preauth-client
branch
from
July 17, 2026 07:50
c9c73c8 to
c2e4d48
Compare
tdiesler
force-pushed
the
ghi50578-preauth-client
branch
from
July 17, 2026 08:57
c2e4d48 to
acf7e4a
Compare
tdiesler
force-pushed
the
ghi50578-preauth-client
branch
from
July 17, 2026 15:46
acf7e4a to
7914057
Compare
closes keycloak#50587 Signed-off-by: Thomas Diesler <tdiesler@proton.me> Decouples pre-authorized credential offers from the creating client by supporting explicit or uniquely discovered target clients. * Adds target_client request support and client discovery. * Updates pre-authorized issuance flows and tests. * Adds offer-state inspection utilities.
tdiesler
force-pushed
the
ghi50578-preauth-client
branch
from
July 17, 2026 19:37
7914057 to
9cea73c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes #50587
Decouples pre-authorized credential offers from the creating client by supporting explicit or uniquely discovered target clients.
depends on