Skip to content

Conversation

@mrts
Copy link
Member

@mrts mrts commented Nov 3, 2025

Backport fixes from main.

WE2-1132

Signed-off-by: Mart Somermaa [email protected]

mrts added 4 commits November 3, 2025 19:03
…cate expiration fixes and CertificateData Optional API from main

WE2-1132

Signed-off-by: Mart Somermaa <[email protected]>
…ectDN() fix from main

WE2-1132

Signed-off-by: Mart Somermaa <[email protected]>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 4, 2025

final WebEidAuthToken token = replaceTokenField(AUTH_TOKEN, "X5C", NEW_MOBILE_ID_CERT);
assertThatThrownBy(() -> validator
.validate(token, VALID_CHALLENGE_NONCE))
// FIXME: should be .isInstanceOf(UserCertificateDisallowedPolicyException.class);
Copy link
Member Author

Choose a reason for hiding this comment

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

In main the error is UserCertificateDisallowedPolicyException as it should be. Double-check why UserCertificateMissingPurposeException is here in v2 and if this needs addressing or not. If not, remove the FIXME.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants