Skip to content
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

fix: use no indentation for json logging #28

Open
wants to merge 1 commit into
base: master
from
Open
Changes from all commits
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

fix: use no indentation for json logging

Enabling indentation for json in logs by setting default json serializer settings causes those logs to be split over multiple entries due to the new line characters. To avoid this the serializer formatting should always be set to override as "None".
  • Loading branch information
mattjhosking committed Mar 31, 2020
commit 822d56d9768ba759d315f8145b8cf92105605d8e
@@ -83,6 +83,7 @@ private static string ToJsonString(JObject loggingInfo)
loggingInfo,
new JsonSerializerSettings
{
Formatting = Formatting.None,
ReferenceLoopHandling = ReferenceLoopHandling.Ignore,
});
}
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.