Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: changing how scripts are loaded from the classpath#45358

Merged
vmuzikar merged 1 commit intokeycloak:mainfrom
shawkins:iss43975
Jan 13, 2026
Merged

fix: changing how scripts are loaded from the classpath#45358
vmuzikar merged 1 commit intokeycloak:mainfrom
shawkins:iss43975

Conversation

@shawkins
Copy link
Contributor

@shawkins shawkins commented Jan 12, 2026

closes: #43975

I'm not sure if this logic didn't age well, or if it wasn't being hit before.

@mabartos
Copy link
Contributor

@shawkins It'd be good to have some automatic tests for this - for this case, it's sufficient if @lhanusov or @stianst verifies it works as expected as the test will be added to the new framework.

@lhanusov
Copy link
Contributor

@mabartos, @shawkins I am running checks on my local machine, once done I will let you know.

@lhanusov
Copy link
Contributor

Embedded server works fine now, augmentation error disappeared, I can confirm, both Junit plugin in IntelliJ and mvn in terminal are good (I also tested distribution mode - which is also good). @shawkins @mabartos @stianst

@mabartos
Copy link
Contributor

I'm not sure if this logic didn't age well, or if it wasn't being hit before.

Yep, wondering why there's no issue in the community - nobody uses that? :P

@shawkins We should probably backport it to 26.5.x.

Maybe this might be slightly related: #44656

@vmuzikar vmuzikar merged commit 819de5e into keycloak:main Jan 13, 2026
85 checks passed
shawkins added a commit to shawkins/keycloak that referenced this pull request Jan 13, 2026
@shawkins
Copy link
Contributor Author

Yep, wondering why there's no issue in the community - nobody uses that? :P

I think it may be related to some cleanup I did around the home dir recently. The check used here before might not have thought to use the classpath files for the embedded server - even though that was the intent of this code.

@shawkins We should probably backport it to 26.5.x.

Opened a PR

mabartos pushed a commit that referenced this pull request Jan 14, 2026
)

closes: #43975


(cherry picked from commit 819de5e)

Signed-off-by: Steve Hawkins <[email protected]>
stianst pushed a commit to stianst/keycloak that referenced this pull request Jan 14, 2026
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.

Test Framework -> Embedded server -> Maven execution failure: Failed to read script file from: scripts/default-policy.js

4 participants