Skip to content

Race condition inside Keycloak build chain execution - #23260

Merged
vmuzikar merged 1 commit into
keycloak:mainfrom
mabartos:raceCondition
Sep 14, 2023
Merged

Race condition inside Keycloak build chain execution#23260
vmuzikar merged 1 commit into
keycloak:mainfrom
mabartos:raceCondition

Conversation

@mabartos

@mabartos mabartos commented Sep 14, 2023

Copy link
Copy Markdown
Member

Fixes #23258

The crypto provider must be set before the Keycloak Session factory initialization.

@keycloak/cloud-native Can you check it, please? Thanks

@mabartos
mabartos requested review from a team as code owners September 14, 2023 14:40

@vmuzikar vmuzikar 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.

LGTM, thanks @mabartos!

@vmuzikar
vmuzikar enabled auto-merge (squash) September 14, 2023 15:09
@vmuzikar
vmuzikar merged commit 2946139 into keycloak:main Sep 14, 2023
@ghost ghost added the flaky-test label Sep 14, 2023
paulbares added a commit to paulbares/keycloak that referenced this pull request Sep 15, 2023
* master: (7978 commits)
  CLI args not recognized when running Quarkus dev mode (keycloak#23254)
  Bump github/codeql-action from 2.21.5 to 2.21.6
  Race condition inside Keycloak build chain execution (keycloak#23260)
  doc(js-providers): Add OIDC object mapper documentation
  Fix NPE by calling delegate in DeploymentDelegate.getAdapterConfig()
  Add new version of Welcome theme based on PatternFly 5 (keycloak#23008)
  Toggle visibility of password input fields in login-ftl-based pages
  i18n: Add Thai language (keycloak#23165)
  Registration page not showing username when edit username is not enabled
  Bump @4tw/cypress-drag-drop from 2.2.4 to 2.2.5 in /js (keycloak#23237)
  Make `minValidity` parameter in `updateToken()` optional (keycloak#23050)
  Bump cypress from 13.1.0 to 13.2.0 in /js (keycloak#23186)
  Add 22.0.3 to release notes (keycloak#23238)
  Removed namespaces from messages (keycloak#23205)
  Send Client ID in token request with JWT Authentication
  Bump @types/tar-fs from 2.0.1 to 2.0.2 in /js (keycloak#23189)
  Bump uuid from 9.0.0 to 9.0.1 in /js (keycloak#23190)
  Bump wireit from 0.13.0 to 0.14.0 in /js (keycloak#23191)
  Bump @playwright/test from 1.37.1 to 1.38.0 in /js (keycloak#23223)
  Bump @testing-library/cypress from 9.0.0 to 10.0.1 in /js (keycloak#23224)
  ...
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.

Race condition inside Keycloak build chain execution

2 participants