Resource leak in TcpChannel when opening and closing TCP Proxies.
scottcnevin opened this issue · 3 comments
In TcpChannel we found that opening and closing TCP Proxies between client and server would leak resources
on the Gen2 heap that were never reclaimed. We have an application that grew to a run-time foot-print of 5 Gb! Running Redgate ANTS memory profiler found a set of objects that were being retained in Gen2 due to a number of references related to the SocketAsyncEventArgs. In the changes, each step in this was able to finally get a flat memory profile when making and breaking connections between client and server.
Love it. Submit the PR and I'll run some tests locally and if everything checks out, I'll get a new version of the package out asap.
Tyler -- not really sure what to do at this point. I opened the PR and made the changes. Not sure how to "submit" the PR at this point on github.
@scottcnevin the PR you created and referenced is into your own fork. As such, I cannot approve that PR nor would it land in the origin in my repo. See the doc:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork