-
Notifications
You must be signed in to change notification settings - Fork 327
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
feat!: Spring Cloud 2024.0 and Spring Boot 3.4.2 #3500
Conversation
This reverts commit 290c7a2.
This reverts commit eaf04dc.
This reverts commit 0fabdbf.
This reverts commit 3eeef1e.
This reverts commit ce40125.
This reverts commit 4015f3f.
this prevents attempts of the form sm@my_secret:default as a whole string
This moves the @RefreshScope annotation to method-level in order to avoid build failures in the native image tests. A good candidate to prove it as good practice is https://github.com/spring-cloud/spring-cloud-commons/blob/c0f3a6190c5e636ac5391371b503f7f1e6e24a8e/docs/modules/ROOT/pages/spring-cloud-commons/application-context-services.adoc?plain=1#L211-L215 The native tests script will also override the refresh scope as it's explained in the docs (see implementation for link).
@@ -1,5 +1,5 @@ | |||
/* | |||
* Copyright 2024 Google LLC | |||
* Copyright 2025 Google LLC |
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.
hm, this is already up to 2025 in main branch. And I notice this change is not in #3411. Can you merge main into this?
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.
Done
|
This pull request introduces an upgrade to Spring Cloud 2024 and Spring Boot 3.4.2
Previously approved (#3411)
Changes per module
sm@secret_id
in addition to the already existingsm://secret_id
format. Users should move away fromsm://
and usesm@
instead. Usage ofsm://
will raise a warning about this being removed in the future. This has been tested usingspring-framework:6.2.2-SNAPSHOT
.Internal changes
QueryMethodEvaluationContextProvider
toValueExpressionDelegate
QueryMethodEvaluationContextProvider
toValueExpressionDelegate