-
Notifications
You must be signed in to change notification settings - Fork 75
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
SB 3.2 - Use Reactor Context-Propagation property #464
Comments
Seems like a good first issue indeed ; we can have a JavaIsoVisitor return |
I think it might have to be a |
Hello, My name is prem and I would like to work on this issue. Is it still active ? |
Hi! Yes sure; if you open up a draft pull request with the tests I'll assign the issue to you. |
We need to replace the use of |
@Shiroo77 |
What problem are you trying to solve?
Use this shiny new property: https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-3.2-Release-Notes#properties
...as a replacement for calling
Hooks.enableAutomaticContextPropagation();
before booting the app, as mentioned in this blog post: https://spring.io/blog/2023/03/30/context-propagation-with-project-reactor-3-unified-bridging-between-reactive#automatic-context-propagationWhat precondition(s) should be checked before applying this recipe?
Probably
UsesType
forHooks
andSpringApplication
Describe the situation before applying the recipe
Describe the situation after applying the recipe
...combined with a spring property addition:
Have you considered any alternatives or workarounds?
Any additional context
Are you interested in contributing this recipe to OpenRewrite?
Maybe eventually; eager contributors are welcome to grab it first
The text was updated successfully, but these errors were encountered: