8000 [OID4VCI]: Fix OID4VCI resource Date headers for conformance and add regression tests by forkimenjeckayang · Pull Request #48277 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

[OID4VCI]: Fix OID4VCI resource Date headers for conformance and add regression tests - #48277

Merged
mposolda merged 1 commit into
keycloak:mainfrom
adorsys:oid4vci-conformace-2
Apr 27, 2026
Merged

mposolda merged 1 commit into
keycloak:mainfrom
adorsys:oid4vci-conformace-2

Conversation

@forkimenjeckayang
@forkimenjeckayang forkimenjeckayang commented Apr 20, 2026 •
Copy link
Copy Markdown
Contributor

Summary

  • Add explicit RFC7231 Date headers to OID4VCI resource responses and issuer metadata endpoints used by conformance checks.
  • Ensure Date header values are emitted in RFC1123 format to satisfy strict conformance validation.
  • Add regression tests in the OID4VC test suite to verify Date header presence and parseability on nonce and well-known issuer endpoints.

Why

The conformance test oid4vci-1_0-issuer-happy-flow failed on CheckForDateHeaderInResourceResponse (RFC7231-7.1.1.2) because resource responses were missing a valid Date header.

closes #48488

See: adorsys#245 (comment)

@keycloak-github-bot
Copy link
Copy Markdown

Unreported flaky test detected

If the flaky tests below are affected by the changes, please review and update the changes accordingly. Otherwise, a maintainer should report the flaky tests prior to merging the PR.

org.keycloak.testsuite.forms.RPInitiatedFrontChannelLogoutTest#testFrontChannelLogoutWithoutSessionRequired

Keycloak CI - Forms IT (firefox)

org.openqa.selenium.TimeoutException: 
Navigation timed out after 10000 ms
Build info: version: '4.28.1', revision: '73f5ad48a2'
System info: os.name: 'Linux', os.arch: 'amd64', os.version: '6.17.0-1010-azure', java.version: '25.0.2'
Driver info: org.openqa.selenium.firefox.FirefoxDriver
...

Report flaky test

@keycloak-github-bot keycloak-github-bot Bot left a comment
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Unreported flaky test detected, please review

@tdiesler
Copy link
Copy Markdown
Contributor

This already works AFAICS

image

@forkimenjeckayang
Copy link
Copy Markdown
Contributor Author

This already works AFAICS

image

Somehow , it's not. I updated the branch which this depends on with latest changes from main ie oid4vci-conformace-1 and ran the test again which fails on oid4vci-1_0-issuer-happy-flow. But the changes on this branch oid4vci-conformace-2 address it

image

…tests

Signed-off-by: forkimenjeckayang <forkimenjeckayang@gmail.com>
@mposolda mposolda left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@mposolda
mposolda merged commit a8aaed2 into keycloak:main Apr 27, 2026
85 checks passed
@Awambeng
Awambeng deleted the oid4vci-conformace-2 branch May 15, 2026 07:05
@stianst stianst mentioned this pull request Jun 1, 2026
sashyo pushed a commit to tide-foundation/keycloak-IGA that referenced this pull request Jul 20, 2026
…tests (keycloak#48277)

closes keycloak#48488


Signed-off-by: forkimenjeckayang <forkimenjeckayang@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[OID4VCI] Fix OID4VCI resource Date headers for conformance

3 participants

0