-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Bump engine dependency to Spring 6 #4869
Comments
Hey @joaquinfelici, can you share some information/documentation about the changes in those PRs? It's hard to understand why they are necessary. |
Hey @mboskamp , discussed with Tassilo who took care of most of this fix. Spring 5 is EOL, so we're changing the default spring version to 6 for the engine. Spring 5 is still supported but must be explicitly overridden. Websphere Liberty and other impacted distros continue using spring 5, hence the exclusions. The JNDI changes are due to a change in Spring, now the testing library for JNDI was removed, and they propose to add |
I left a few comments/questions in the PRs. |
Context
The engine currently has a dependency on Spring 5 to allow configuring the engine via an XML file.
Acceptance Criteria (Required on creation)
The dependency is updated to Spring 6.
Hints
Links
Breakdown
Pull Requests
spring-beans
to 6 #4862Dev2QA handover
The text was updated successfully, but these errors were encountered: