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

Guard against multiple body subscriptions in Jetty and JDK reactive responses #32100

Closed
poutsma opened this issue Jan 24, 2024 · 0 comments
Closed

Guard against multiple body subscriptions in Jetty and JDK reactive responses #32100

poutsma opened this issue Jan 24, 2024 · 0 comments
Assignees
@poutsma
Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) status: backported An issue that has been backported to maintenance branches type: bug A general bug
Milestone
6.1.4

Comments

@poutsma
Copy link
Contributor

poutsma commented Jan 24, 2024

Unlike the HttpComponentsClientHttpResponse, we do not guard against subscribing to the response body multiple times in the JdkClientHttpResponse and JettyClientHttpResponse.
@poutsma poutsma added in: web Issues in web modules (web, webmvc, webflux, websocket) type: bug A general bug labels Jan 24, 2024
@poutsma poutsma added this to the 6.1.4 milestone Jan 24, 2024
@poutsma poutsma self-assigned this Jan 24, 2024
@poutsma poutsma mentioned this issue Jan 24, 2024
Closed
@poutsma poutsma added for: backport-to-5.3.x Marks an issue as a candidate for backport to 5.3.x for: backport-to-6.0.x Marks an issue as a candidate for backport to 6.0.x labels Jan 24, 2024
@github-actions github-actions bot added status: backported An issue that has been backported to maintenance branches and removed for: backport-to-5.3.x Marks an issue as a candidate for backport to 5.3.x labels Jan 24, 2024
@github-actions github-actions bot mentioned this issue Jan 24, 2024
Closed
@github-actions github-actions bot removed the for: backport-to-6.0.x Marks an issue as a candidate for backport to 6.0.x label Jan 24, 2024
@github-actions github-actions bot mentioned this issue Jan 24, 2024
Closed
poutsma added a commit that referenced this issue Jan 24, 2024
@poutsma
Guard against multiple body subscriptions
c749a14 
Before this commit, the JDK and Jetty connectors do not have any
safeguards against multiple body subscriptions. Such as check has now
been added.

See gh-32100
Closes gh-32102
poutsma added a commit that referenced this issue Jan 24, 2024
@poutsma
Guard against multiple body subscriptions
186d6e3 
Before this commit, the Jetty connector did not have any
safeguard against multiple body subscriptions. Such as check has now
been added.

See gh-32100
Closes gh-32101
@poutsma poutsma mentioned this issue Apr 29, 2024
Closed
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) status: backported An issue that has been backported to maintenance branches type: bug A general bug
Projects
None yet
Milestone
6.1.4
Development

No branches or pull requests
1 participant
@poutsma