spring-cloud-config icon indicating copy to clipboard operation
spring-cloud-config copied to clipboard

Published 20 hours ago • verified publisher icon spring-cloud

[Refactoring] Introduce RequestContext to easily pass new parameters to downstream methods

Open opeco17 opened this issue 10 months ago • 4 comments

Fixes https://github.com/spring-cloud/spring-cloud-config/issues/2407

opeco17 avatar Apr 22 '24 13:04 opeco17

Hi, @ryanjbaxter @spencergibb

Could you please review this PR? Thank you!

opeco17 avatar Apr 28 '24 03:04 opeco17

I think this is the right idea for where we want to go.

The issue is as is it could only go in a major release because of the breaking API changes. That is fine but after talking about it some more with the team we should really be deprecating the existing public APIs in the 2024 release (which will be a minor) and then remove them in 2025 release (which will likely be our next major). Its more of a process thing than anything else, but this way we could also get #2402 in the 2024 release as well.

ryanjbaxter avatar Apr 29 '24 19:04 ryanjbaxter

@ryanjbaxter

Thank you for sharing. It would be nice if #2402 becomes available in 2024 release.

May I know the next action that I should take? Is some revise to this PR necessary?

opeco17 avatar Apr 29 '24 23:04 opeco17

Yes we would need to keep the existing public APIs intact and unchanged and mark them all as @Deprecated then introduce the new ones (as you already have in this PR) which use the RequestContext object.

ryanjbaxter avatar Apr 30 '24 13:04 ryanjbaxter