Graphite.NET icon indicating copy to clipboard operation
Graphite.NET copied to clipboard

TCP Client swallows errors

Open ProTip opened this issue 10 years ago • 0 comments

Unfortunately this little bit here makes the client unusable for me as-is: https://github.com/ragnard/Graphite.NET/blob/master/Graphite/GraphiteTcpClient.cs#L36-L39

I would recommend removing the entire try/catch and let the consumer deal with errors as a quick fix. I have done this and I just throw away the client and create a new one if an error comes up(disconnected tcp client, unable to connect, etc).

ProTip avatar Jan 29 '15 00:01 ProTip