Skip to content

[release/8.0] [SignalR] [Java] Fix hang when reading an HTTP response body fails - #69209

Merged
wtgodbe merged 1 commit into
release/8.0from
backport/pr-69037-to-release/8.0
Sep 11, 2026
Merged

[release/8.0] [SignalR] [Java] Fix hang when reading an HTTP response body fails#69209
wtgodbe merged 1 commit into
release/8.0from
backport/pr-69037-to-release/8.0

Conversation

@github-actions

@github-actions github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Backport of #69037 to release/8.0

/cc @BrennanConroy

[SignalR] [Java] Fix hang when reading an HTTP response body fails

Description

If a HubConnection is starting and the client has a network glitch or loses network connection, it can leave the HubConnection in a hanging state and they would need to restart the app to reset the state.

Customer Impact

1P is hitting this in production. Sometimes the client app hangs and they don't have a way to not make it hang. We've given a workaround of cleaning up the OkHttp state and discarding the HubConnection. But it's a very ugly and temporary workaround.

Regression?

  • Yes
  • No

Risk

  • High
  • Medium
  • Low

Simple change, small scope and easy test coverage

Verification

  • Manual (required)
  • Automated

Packaging changes reviewed?

  • Yes
  • No
  • N/A

@BrennanConroy BrennanConroy added Servicing-consider Shiproom approval is required for the issue area-signalr Includes: SignalR clients and servers labels Sep 10, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Hi @github-actions[bot]. Please make sure you've updated the PR description to use the Shiproom Template. Also, make sure this PR is not marked as a draft and is ready-to-merge.

To learn more about how to prepare a servicing PR click here.

@rbhanda rbhanda added Servicing-approved Shiproom has approved the issue and removed Servicing-consider Shiproom approval is required for the issue labels Sep 10, 2026
@rbhanda rbhanda added this to the 8.0.32 milestone Sep 10, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Hi @github-actions[bot]. This PR was just approved to be included in the upcoming servicing release. Somebody from the @dotnet/aspnet-build team will get it merged when the branches are open. Until then, please make sure all the CI checks pass and the PR is reviewed.

@wtgodbe

wtgodbe commented Sep 11, 2026

Copy link
Copy Markdown
Member

Failures unrelated

@wtgodbe
wtgodbe merged commit 6d7a8e9 into release/8.0 Sep 11, 2026
24 of 27 checks passed
@wtgodbe
wtgodbe deleted the backport/pr-69037-to-release/8.0 branch September 11, 2026 21:50
@dotnet-policy-service dotnet-policy-service Bot removed this from the 8.0.32 milestone Sep 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-signalr Includes: SignalR clients and servers Servicing-approved Shiproom has approved the issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants