Taras Kozlov
Taras Kozlov
> According to https://docs.python.org/3/library/logging.html#logrecord-attributes, the default date format should have 3 digits instead of 6 for the millisecond portion of the time. Fixed
> > > According to https://docs.python.org/3/library/logging.html#logrecord-attributes, the default date format should have 3 digits instead of 6 for the millisecond portion of the time. > > > > > >...
Well, it is not an encoding issue, it is an attempt to print an uninitialized buffer here: https://github.com/microsoft/picologging/blob/e648cb336d1f86053af342129c7d170672a76886/src/picologging/formatter.cxx#L101 If you're lucky it doesn't crash python, but on a bad day...
I made a PR that fixes the issue https://github.com/microsoft/picologging/pull/208
Implemented in https://github.com/microsoft/picologging/pull/209
> We use C++'s `strftime` function for performance reasons, which doesn't support microseconds. > > I can look at adding this feature Hi Tony, would you mind having a quick...
> @microsoft-github-policy-service agree @microsoft-github-policy-service agree
Hi @tonybaloney. Would you be able to take a look at my PRs please? Do you think you will a make new release any time soon?
> The other PR is merged so I think this one can be closed? Sorry for confusion. The other PR was just fixing default asctime formatting. This one adds support...
@tonybaloney It is ready