Skip to content

fix: properly map name and description in JwtToVerifiableCredentialTransformer#5787

Merged
paullatzelsperger merged 1 commit into
eclipse-edc:mainfrom
paullatzelsperger:fix/properly_map_name_and_description
May 28, 2026
Merged

fix: properly map name and description in JwtToVerifiableCredentialTransformer#5787
paullatzelsperger merged 1 commit into
eclipse-edc:mainfrom
paullatzelsperger:fix/properly_map_name_and_description

Conversation

@paullatzelsperger
Copy link
Copy Markdown
Member

@paullatzelsperger paullatzelsperger commented May 28, 2026

Summary

  • JwtToVerifiableCredentialTransformer was incorrectly setting name to the JWT sub claim instead of reading it from the VC object
  • Now both name and description are read directly from the vc payload properties
  • Added tests covering the new behaviour (name+description present, name absent, description absent)

Contributes to eclipse-edc/IdentityHub#701

…ableCredentialTransformer

Previously, the transformer incorrectly set name to the JWT subject claim.
Now name and description are read directly from the vc object properties,
and the JWT subject is no longer used as a fallback for name.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@paullatzelsperger paullatzelsperger requested a review from a team as a code owner May 28, 2026 08:26
@paullatzelsperger paullatzelsperger changed the title fix: properly map name and description from VC payload in JwtToVerifiableCredentialTransformer fix: properly map name and description in JwtToVerifiableCredentialTransformer May 28, 2026
@paullatzelsperger paullatzelsperger added the bug Something isn't working label May 28, 2026
@paullatzelsperger paullatzelsperger merged commit 71edf2a into eclipse-edc:main May 28, 2026
42 of 49 checks passed
@paullatzelsperger paullatzelsperger deleted the fix/properly_map_name_and_description branch May 28, 2026 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants