fixed: Sockets hanging (closes #7097)

DevExpress / testcafe-hammerhead

A powerful web-proxy used as a core for the TestCafe testing framework. :hammer: :smiley:

https://testcafe.io

MIT License

171 stars 161 forks source link

fixed: Sockets hanging (closes #7097) #2976

Closed PavelMor25 closed 10 months ago

PavelMor25 commented 10 months ago

Purpose

Due to changes in Node.js, the old approach of destroying the request is no longer suitable, as the check within the method itself determines that it has already been destroyed.

Approach

We have added a closeConnectionOnError method in RequestPipelineContext that bypasses this check.

References

issue 7097

Pre-Merge TODO

[ ] Write tests for your proposed changes
[ ] Make sure that existing tests do not fail