Skip to content

[OID4VCI] Remove signed metadata toggle from admin UI#47515

Merged
mposolda merged 4 commits intokeycloak:mainfrom
adorsys:issue-47476
Mar 27, 2026
Merged

[OID4VCI] Remove signed metadata toggle from admin UI#47515
mposolda merged 4 commits intokeycloak:mainfrom
adorsys:issue-47476

Conversation

@Ogenbertrand
Copy link
Copy Markdown
Contributor

@Ogenbertrand Ogenbertrand commented Mar 27, 2026

This PR aligns the OID4VCI admin UI with the backend decision to remove oid4vci.signed_metadata.enabled.

It:

  • removes the signed metadata toggle from Realm Settings -> Tokens -> OID4VCI attributes
  • always shows Signed Metadata Lifespan
  • always shows Signed Metadata Signing Algorithm
  • updates the Playwright coverage to verify the toggle is absent, the signed metadata fields are visible by default, and oid4vci.signed_metadata.enabled is no longer persisted

Closes: #47476

@Ogenbertrand Ogenbertrand requested review from a team as code owners March 27, 2026 09:55
@mposolda mposolda self-assigned this Mar 27, 2026
@mposolda
Copy link
Copy Markdown
Contributor

LGTM as long as tests are OK.

Just 2 minor things:

  • DCO should be fixed (by adding signature to the commit)
  • Will be good to remove the label and tooltip signedMetadataLifespan and signedMetadataLifespanHelp as it seems those are not used anymore

@Ogenbertrand
Copy link
Copy Markdown
Contributor Author

LGTM as long as tests are OK.

Just 2 minor things:

  • DCO should be fixed (by adding signature to the commit)
  • Will be good to remove the label and tooltip signedMetadataLifespan and signedMetadataLifespanHelp as it seems those are not used anymore

I think there may be a small mix-up in the key names. signedMetadataLifespan and signedMetadataLifespanHelp are still used by the lifespan field in TokensTab.tsx, so removing them would break the label/help text for that control. The keys that became unused with this change are signedIssuerMetadata and signedIssuerMetadataHelp, which belonged to the removed toggle @mposolda.

@mposolda
Copy link
Copy Markdown
Contributor

LGTM as long as tests are OK.
Just 2 minor things:

  • DCO should be fixed (by adding signature to the commit)
  • Will be good to remove the label and tooltip signedMetadataLifespan and signedMetadataLifespanHelp as it seems those are not used anymore

I think there may be a small mix-up in the key names. signedMetadataLifespan and signedMetadataLifespanHelp are still used by the lifespan field in TokensTab.tsx, so removing them would break the label/help text for that control. The keys that became unused with this change are signedIssuerMetadata and signedIssuerMetadataHelp, which belonged to the removed toggle @mposolda.

Yes, you are right. I am sorry for the confusion. I meant to remove those, which are unused ( signedIssuerMetadata and signedIssuerMetadataHelp)

@mposolda
Copy link
Copy Markdown
Contributor

@Ogenbertrand Could you please rebase on top of latest main? The build error should be fixed already.

Copy link
Copy Markdown
Contributor

@mposolda mposolda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Ogenbertrand Thanks!

@mposolda mposolda merged commit f9a1016 into keycloak:main Mar 27, 2026
63 checks passed
@github-actions
Copy link
Copy Markdown

Translation Changes Notification

Language maintainers: Please review the translation changes in your language in Weblate.

The following languages have been updated in this PR:

Spanish (es)

@herver1971, @anthieni

Changes waiting for approval.

French (fr)

@Dodouce, @GitSpoon

Changes waiting for approval.

Russian (ru)

@petrov9, @pasternake

Changes waiting for approval.

Swedish (sv)

@yeager, @haaninjo

Changes waiting for approval.

Turkish (tr)

@spctr, @ariferol

Changes waiting for approval.

Simplified Chinese (zh_Hans)

@jasonqsong, @charliedcc

Changes waiting for approval.

Traditional Chinese (zh_Hant)

@allen0099, @benwater12

Changes waiting for approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[OID4VCI] SIGNED_METADATA_ENABLED_ATTR from OID4VCI admin UI and always show signed metadata settings

2 participants