fix: revert the order of stream closure and disconnect

[chanseokoh]

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

• Make sure to open an issue as a bug/issue: "Premature end of chunk coded message body" and TruncatedChunkException when calling HttpResponse.disconnect() #1409
• Ensure the tests and linter pass
• Code coverage does not decrease (if any source code was changed)
• Appropriate docs were updated (if necessary)

Fixes #1409 ☕️

I don't know how exactly #1303 should be resolved, but given the critical nature of #1409, we should revert #1315 now.

[chanseokoh]

@chingor13

[lesv]

Ping - @chanseokoh Since it's been almost a month, could you confirm what you want to want (and in what order) you want them to happen here? I see:

• revert 1315
• merge this PR

[lesv]

It looks like you are reverting a test from #1315 - so do you still need the whole PR reverted?

[chanseokoh]

@lesv you mean #1315, not #1305, right? I am not commit-reverting it, since #1315 has some other refactoring stuff we want to keep. This PR just reverts what's necessary, i.e, the order of stream closure and disconnect. Merging this PR will make everything right.