🔥 Do not detaches sockets / terminates client stream

[AlexV525]

Fixes #2170

New Pull Request Checklist

• I have read the Documentation
• I have searched for a similar pull request in the project and found none
• I have updated this branch with the latest main branch to avoid conflicts (via merge from master or rebase)
• I have added the required tests to prove the fix/feature I'm adding
• I have updated the documentation (if necessary)
• I have run the tests without failures
• I have updated the CHANGELOG.md in the corresponding package

[AlexV525]

I've failed to write tests for these changes. I've tried using regular implementation and also Mockito, the detachSocket or terminate happens after the current event loop of the request. Now we stop calling them means no innovations. And the exception requires some duration gap which I cannot find during my tests writing.

[AlexV525]

I'm probably running into the state that some contents are using the main ref. The test works well locally.

[kuhnroyal]

Looks reasonable, we should find a reproducible test case before we merge this imo.

[AlexV525]

Looks reasonable, we should find a reproducible test case before we merge this imo.

I'd appreciate any ideas. I still cannot find a solid case unless we can make some integration tests.

[AlexV525]

Oh babe, coding is such a poetic act that can only be solved if you have feelings.

I've added relevant tests and all passed locally.

dio	http2

[github-actions]

Code Coverage Report

Package	Base Coverage	New Coverage	Difference
dio	🟢 80.26%	🟢 81.24%	🟢 0.98%
Overall Coverage	🟢 80.26%	🟢 81.24%	🟢 0.98%

Minimum allowed coverage is 0%, this run produced 81.24%