-
Notifications
You must be signed in to change notification settings - Fork 922
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Serilog sample to ref servers #1263
Conversation
mregen
commented
Jan 28, 2021
- the Serilog trace event logger replaces the default log output file
- optional logging to console / debug
Codecov Report
@@ Coverage Diff @@
## master #1263 +/- ##
=========================================
Coverage ? 41.80%
=========================================
Files ? 289
Lines ? 104934
Branches ? 0
=========================================
Hits ? 43868
Misses ? 61066
Partials ? 0 Continue to review full report at Codecov.
|
if (lastContact) | ||
{ | ||
item += String.Format("Last Event:{0:HH:mm:ss}", session.SessionDiagnostics.ClientLastContactTime.ToLocalTime()); | ||
item += string.Format("Last Event:{0:HH:mm:ss}", session.SessionDiagnostics.ClientLastContactTime.ToLocalTime()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is better to use StringBuilder instead of string concatenation by +=
- the Serilog trace event logger replaces the default log output file - optional logging to console / debug