Skip to content

preserve unread data across detach#1

Closed
interface95 wants to merge 3 commits into
masterfrom
fix-kestrel-detach-handoff
Closed

preserve unread data across detach#1
interface95 wants to merge 3 commits into
masterfrom
fix-kestrel-detach-handoff

Conversation

@interface95

Copy link
Copy Markdown
Owner

Summary

  • Preserve unread bytes after a yielded package so detach handoff does not let the old pipeline consume raw data from the same buffer.
  • Add regression coverage for detach preserving buffered data after the current package.
  • Add a Kestrel HTTP/2 detach demo with no-signal-wait, stress, and TCP comparison modes.

Verification

  • dotnet test test/SuperSocket.Tests/SuperSocket.Tests.csproj --filter FullyQualifiedName~PipeConnectionBaseTest --logger "console;verbosity=minimal"
  • dotnet test test/SuperSocket.Tests/SuperSocket.Tests.csproj --filter FullyQualifiedName~ClientTest.TestDetachableConnection --logger "console;verbosity=detailed"
  • dotnet build -c Release
  • dotnet test test/SuperSocket.Tests/SuperSocket.Tests.csproj --no-build --logger "console;verbosity=minimal"
  • dotnet run --project samples/KestrelHttp2DetachDemo/KestrelHttp2DetachDemo.csproj -c Release -- --no-signal-wait
  • dotnet run --project samples/KestrelHttp2DetachDemo/KestrelHttp2DetachDemo.csproj -c Release -- --compare-tcp
  • dotnet run --project samples/KestrelHttp2DetachDemo/KestrelHttp2DetachDemo.csproj -c Release -- --stress-early-raw

claude and others added 3 commits May 26, 2026 00:39
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@interface95

Copy link
Copy Markdown
Owner Author

Closing this fork-internal PR because the intended upstream PR is kerryjiang#831.

@interface95 interface95 reopened this May 26, 2026
@interface95

Copy link
Copy Markdown
Owner Author

Closing again after using fork Actions for upstream PR verification. Upstream PR remains kerryjiang#831.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants