Enhance logging - add millisecs (add %f specifier for timestamp-format) #1087
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Found by: michaelortmann
Patch by: michaelortmann
Fixes:
One-line summary:
Add %f specifier for timestamp-format
Additional description (if needed):
I always wanted to have the possibility to let eggdrop log showing milliseconds. Often i found myself adding this while debugging. Last time i missed it was with ident race debugging. This PR adds %f as an additional format specifier for timestamp-format.
@geo: you probaly wanna add some doc for it.
Test cases demonstrating functionality (if applicable):
conf:
set timestamp-format {[%H:%M:%S.%f]}
Test
%f
vs.%%f
:conf:
set timestamp-format {[%H:%M:%S.%f]}:
result:
[12:57:54.481]
conf:
set timestamp-format {[%%H:%%M:%%S.%%f]}
result:
[%H:%M:%S.%f]