Skip to content

Conversation

@BrennanConroy
Copy link
Member

Fixes #46861
Regressed by #41180

@BrennanConroy BrennanConroy added the area-signalr Includes: SignalR clients and servers label Feb 24, 2023
@ghost
Copy link

ghost commented Mar 4, 2023

Looks like this PR hasn't been active for some time and the codebase could have been changed in the meantime.
To make sure no breaking changes are introduced, please leave an /azp run comment here to rerun the CI pipeline and confirm success before merging the change.

@ghost ghost added the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Mar 4, 2023
@BrennanConroy
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@ghost ghost removed the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Mar 8, 2023
@BrennanConroy
Copy link
Member Author

/backport to release/7.0

@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2023

Started backporting to release/7.0: https://github.com/dotnet/aspnetcore/actions/runs/4367264350

@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2023

@BrennanConroy backporting to release/7.0 failed, the patch most likely resulted in conflicts:

$ git am --3way --ignore-whitespace --keep-non-patch changes.patch

Applying: Fix canceling flush of CancelInvocation when canceling server-to-client stream
Using index info to reconstruct a base tree...
M	src/SignalR/clients/csharp/Client.Core/src/HubConnection.cs
Falling back to patching base and 3-way merge...
Auto-merging src/SignalR/clients/csharp/Client.Core/src/HubConnection.cs
CONFLICT (content): Merge conflict in src/SignalR/clients/csharp/Client.Core/src/HubConnection.cs
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Fix canceling flush of CancelInvocation when canceling server-to-client stream
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2023

@BrennanConroy an error occurred while backporting to release/7.0, please check the run log for details!

Error: git am failed, most likely due to a merge conflict.

@BrennanConroy BrennanConroy merged commit 4e9a3e8 into main Mar 8, 2023
@BrennanConroy BrennanConroy deleted the brecon/cancelStream branch March 8, 2023 22:22
@ghost ghost added this to the 8.0-preview3 milestone Mar 8, 2023
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[SignalR] Server doesn't stop streaming after cancellation on client

3 participants