{{Quickfixn}} ClientHandlerThread Debug Logging

[James Coverdale]

Perfect, PR has been submitted, let me know your thoughts.

https://github.com/connamara/quickfixn/pull/705

Thanks,
James.

From: Quickfixn <quickfixn-bounces at lists.quickfixn.com> On Behalf Of Grant Birchmeier
Sent: 09 September 2021 14:55
To: Mailing list for QuickFIX/n <quickfixn at lists.quickfixn.com>
Subject: Re: {{Quickfixn}} ClientHandlerThread Debug Logging

That sounds like a good feature.  I would welcome a PR on it.

On Thu, Sep 9, 2021 at 3:13 AM James Coverdale <jcoverdale at redev.co.uk<mailto:jcoverdale at redev.co.uk>> wrote:
Hi,

We have run into issues with large numbers of ‘ClientHandlerThread’ debug logs being created, seems that other people have had the same issue: https://github.com/connamara/quickfixn/issues/684

It doesn’t seem like a major change to add a config bool to have the ability to turn these ‘debug’ logs off in production environments? Update ClientHandlerThread.cs to use an ILOG for the ‘log_’ and a NullLogger if the new config bool is set.

I can create a PR, but wanted to run the idea past people first.

Thanks,
James.


_______________________________________________
Quickfixn mailing list
Quickfixn at lists.quickfixn.com<mailto:Quickfixn at lists.quickfixn.com>
http://lists.quickfixn.com/listinfo.cgi/quickfixn-quickfixn.com


--
Grant Birchmeier
Connamara Systems, LLC
Made-To-Measure Trading Solutions.
Exactly what you need. No more. No less.
http://connamara.com

This email, along with any attachments, is confidential. If you believe you received this message in error, please contact the sender immediately and delete all copies of the message. Thank you from Connamara Systems, LLC.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.quickfixn.com/pipermail/quickfixn-quickfixn.com/attachments/20210910/fdd975ea/attachment.htm>