Skip to content

Use Optional instead of getOrDefault for settings in testSMTPConnection (26.2) - #41719

Merged
ahus1 merged 1 commit into
keycloak:release/26.2from
rmartinc:backport-41661-26.2
Aug 8, 2025
Merged

ahus1 merged 1 commit into
keycloak:release/26.2from
rmartinc:backport-41661-26.2

Conversation

@rmartinc

@rmartinc rmartinc commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

Closes #41643

PR: #41661
Commit: a58556d
PR branch: backport-41661-26.2
Target branch: https://github.com/keycloak/keycloak/tree/release/26.2

@rmartinc
rmartinc requested review from a team as code owners August 7, 2025 12:04
@rmartinc
rmartinc force-pushed the backport-41661-26.2 branch from 11d8512 to 30649cd Compare August 7, 2025 13:21
@rmartinc
rmartinc marked this pull request as draft August 7, 2025 13:34
@rmartinc

rmartinc commented Aug 7, 2025 •

Copy link
Copy Markdown
Contributor Author

It seems that there are issues in the JS CI. If the last node is used 22.18.0 there are errors in the tests (see https://github.com/keycloak/keycloak/actions/runs/16804510625/job/47593956118?pr=41719). Chatting with @jonkoops, commented me to test with the previous version and it seems that is working. So we have to use the previous node version or add all the file extensions in those imports.

Finally this issue also affects main, so the JS CI failed because of #41728

@rmartinc
rmartinc force-pushed the backport-41661-26.2 branch from 30649cd to 594af71 Compare August 7, 2025 14:28
@rmartinc
rmartinc marked this pull request as ready for review August 7, 2025 14:30
Closes keycloak#41643

Signed-off-by: rmartinc <rmartinc@redhat.com>
(cherry picked from commit a58556d)
@rmartinc
rmartinc force-pushed the backport-41661-26.2 branch from 594af71 to b41f579 Compare August 8, 2025 12:45
@rmartinc

rmartinc commented Aug 8, 2025

Copy link
Copy Markdown
Contributor Author

Rebase to incorporate the fix for node fix.

@ahus1
ahus1 merged commit 3f669f8 into keycloak:release/26.2 Aug 8, 2025
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