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

"multiple subscribers not supported" when using WebClient exchange #32728

Closed
github-actions bot opened this issue Apr 29, 2024 · 4 comments
Closed

"multiple subscribers not supported" when using WebClient exchange #32728

github-actions bot opened this issue Apr 29, 2024 · 4 comments
Assignees
@poutsma
Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) type: backport An issue that is a backport of another issue to a maintenance branch type: regression A bug that is also a regression
Milestone
5.3.35

Comments

@github-actions
Copy link
Contributor

Backport of gh-32727
@github-actions github-actions bot added in: web Issues in web modules (web, webmvc, webflux, websocket) type: backport An issue that is a backport of another issue to a maintenance branch type: regression A bug that is also a regression labels Apr 29, 2024
@github-actions github-actions bot added this to the 5.3.35 milestone Apr 29, 2024
@poutsma poutsma self-assigned this Apr 29, 2024
@poutsma
Copy link
Contributor

poutsma commented Apr 29, 2024

After more investigation, it appears that a 5.3.x back port is not necessary, see #32727 (comment).

@poutsma poutsma closed this as not planned Won't fix, can't repro, duplicate, stale Apr 29, 2024
@poutsma poutsma added the status: invalid An issue that we don't feel is valid label Apr 29, 2024
@poutsma poutsma removed this from the 5.3.35 milestone Apr 29, 2024
@poutsma
Copy link
Contributor

poutsma commented Apr 30, 2024

Reopened, see #32727 (comment) for the reason.

@poutsma poutsma reopened this Apr 30, 2024
@poutsma poutsma changed the title Revert guard against multiple body subscriptions for Jetty Reactive Client < 4.x "multiple subscribers not supported" when using WebClient exchange Apr 30, 2024
@poutsma poutsma added this to the 5.3.35 milestone Apr 30, 2024
poutsma added a commit that referenced this issue Apr 30, 2024
@poutsma
Fix guard against multiple subscriptions
b3724eb 
This commit changes the guard against multiple subscriptions, as the
previously used doOnSubscribe hook could not function as guard in
certain scenarios.

See gh-32727
Closes gh-32728
@github-actions GitHub Actions
Copy link
Contributor Author

Fixed via b3724eb

@jhoeller jhoeller removed this from the 5.3.35 milestone Jul 10, 2024
@izeye
Copy link
Contributor

izeye commented Jul 20, 2024

This seems to have been removed from the 5.3.35 milestone accidentally.

The release notes for 5.3.35 includes this, and v5.3.35 includes the referenced commit and the changes.

I guess the remaining "status: invalid" label confused @jhoeller.

@bclozel bclozel removed the status: invalid An issue that we don't feel is valid label Jul 20, 2024
@bclozel bclozel added this to the 5.3.35 milestone Jul 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@poutsma poutsma

Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) type: backport An issue that is a backport of another issue to a maintenance branch type: regression A bug that is also a regression
Projects
None yet
Milestone
5.3.35
Development

No branches or pull requests
4 participants
@bclozel @poutsma @izeye @jhoeller