-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/no gradle #44
Conversation
eba0849
to
ee7f6d3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leaving some comments/questions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you 🚀
Thanks for the review @maxdanilov, those changes has just been published in https://github.com/camunda/keycloak/releases/tag/2024-03-26-001 |
Previously, we maintained a gradle.build file with dependency versions. These dependencies could be updated regularly.
Since version 24 of Keycloak, the Keycloak team has been testing the integration of Aurora DB with a version of the AWS wrapper referenced here: keycloak/keycloak@eadd1c4
To align with what is tested by Keycloak and thus avoid compatibility issues, this PR makes a modification to the dependency construction.
It retrieves the pom.xml of the Keycloak version, extracts the referenced version of the AWS wrapper, and then downloads the pom.xml of the latter and installs all transitive dependencies with Maven (this is done in the builder step).
A utility script retrieves the version of the wrapper to download from the version of keycloak.
This PR also includes the removal of the launcher workaround since bitnami/containers#63945 has been merged.
fixes #43