Skip to content
This repository has been archived by the owner on Oct 8, 2021. It is now read-only.

TcpHandler is spamming in Console.Write #5

Open
deniszykov opened this issue Mar 20, 2016 · 6 comments
Open

TcpHandler is spamming in Console.Write #5

deniszykov opened this issue Mar 20, 2016 · 6 comments

Comments

@deniszykov
Copy link

It would be polite for the NetGain do not write logs directly to console.
For module with so many abstractions, it is strange that that logging is done through the console.

@mgravell
Copy link
Contributor

That's fair. Will revisit.
On 20 Mar 2016 8:40 a.m., "deniszykov" [email protected] wrote:

It would be polite for the NetGain do not write logs directly to console.

For module with so many abstractions, it is strange that that logging is
done through the console.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#5

@mleenhardt
Copy link

Hey @mgravell, thanks for that awesome project! Has there been any progress on this? We're interested in using the lib but having hardcoded Console.Writeline isn't an option for us and we would ideally like to not fork the project just to fix this. #10 seems like a decent alternative while other .net core logging discussion are happening. Would you consider it meanwhile?

@mgravell
Copy link
Contributor

Yes; I have some incomplete changes to push on this - I will look at the logging at the same time

@mgravell
Copy link
Contributor

K, so I've uploaded https://www.nuget.org/packages/StackExchange.NetGain/1.0.39 which makes ErrorLog and Log available on the handler or service; they default to Console.Out or Console.Error to be equivalent, but you can either null them out or provide your own.

@mleenhardt
Copy link

Thank you! The updated package currently shows as unlisted and won't by picked up by a nuget update, just making sure that's intended. I'll manually download it for now. 😃

@mgravell
Copy link
Contributor

Yes that was intended. I didn't have the energy to change the build server
to get it marked as beta. I'm bad.

On Fri, 18 Nov 2016 17:55 Mathieu Leenhardt, [email protected]
wrote:

Thank you! The updated package currently shows as unlisted and won't by
picked up by a nuget update, just making sure that's intended. I'll
manually download it for now. 😃


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#5 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABDsCCxych0-V7IojEI7cvu0WdzDhpSks5q_eaJgaJpZM4H0n4L
.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants