Adi Roiban

Thanks for the review. I am waiting 1 week, as requested by the procees, and then will merge. The process requires the approval of 3 contributors ... but given that...

Thanks for your reviews. I have enabled auto-merge on this PR. I hope it will land soon.

Hi Ali. Thanks for your feedback. Twisted is an open source project, that is as good as the contribution from the volunteers. If you have the time and knowledge and...

Thanks Vivek for the report. Can you please share the code of your application that is triggering the memory leak? A short, [self contained example](http://www.sscce.org/) would help a lot. Regards

Is this commit working for you ? https://github.com/twisted/twisted/commit/df9d19cf708a7157411b243264c2bea2b799734c If you can pinpoint the commit in trunk that introduced the error, that can help. From what I cna see https://github.com/twisted/twisted/commit/64506ce3bb578d708372237872aa6b8b90890cb6 is...

Perfect. So the regression was introduced in df9d19c I think that we have a better starting point for troubleshooting :)

Hi Vivek. This is a community driver open source project. As a community, it is expected that the person affected by an issue is also working to fix it. The...

Do we want to fix this, or just deprecate the code?

Hi Torben see also https://github.com/twisted/twisted/issues/5379 and https://github.com/twisted/twisted/issues/3621 and https://github.com/twisted/twisted/issues/6928 ... there are a lot of related issued :( Implementing streaming for HTTP server-side requests required significant changes One of the...

+1 deprecate it... and I think that we should just skip trying to fix the [existing bugs](https://github.com/twisted/twisted/issues?q=is%3Aissue+is%3Aopen+HTTPClient )