Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Support latest project reactor version 3.5.0 #7107

Closed
trask opened this issue Nov 8, 2022 · 1 comment
Closed

Support latest project reactor version 3.5.0 #7107

trask opened this issue Nov 8, 2022 · 1 comment

Comments

@trask
Copy link
Member

trask commented Nov 8, 2022

And revert muzzle limits added in #7106
@trask trask mentioned this issue Nov 8, 2022
Merged
trask added a commit that referenced this issue Nov 8, 2022
@trask
Add muzzle limits to fix CI (#7106)
d40377b 
~I'll create a tracking issue to remove these and support new versions.~

Tracking issue added to support latest project reactor 3.5.0 and revert
these limits: #7107
@trask trask modified the milestones: v1.20.0, v1.21.0 Nov 12, 2022
@trask trask modified the milestones: v1.21.0, v1.22.0 Dec 12, 2022
@trask trask mentioned this issue Jan 10, 2023
Merged
trask added a commit that referenced this issue Jan 12, 2023
@trask
Use new reactor contextWrite when available (from reactor 3.4.0) (#7538)
09b63d2 
Related to #7107 and #7202

Support WebFlux 6. 

Supporting reactor 3.5 seems pretty straightforward, the
`subscriberContext()` was deprecated in 3.4 in favor of
`contextWrite()`. In 3.5, `subscriberContext()` was removed.

This PR doesn't bump `latestDepTestLibrary` to 3.5 yet because there are
a couple of tests that succeed in 3.4 using `contextWrite()` but fail in
3.5 using `contextWrite()`.

My proposal is to review/merge this PR, and then I can ping our resident
reactor experts to see if they have thoughts on the failing tests in
3.5.
@trask trask modified the milestones: v1.22.0, v1.23.0 Jan 12, 2023
@trask trask removed this from the v1.23.0 milestone Feb 14, 2023
@trask
Copy link
Member Author

trask commented Dec 23, 2024

Resolved by #11235

@trask trask closed this as completed Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@trask