Skip to content

Add leading zeros to the milliseconds part of a log timestamp - #13

Merged
smirnoal merged 1 commit into
aws:mainfrom
smirnoal:main
Mar 29, 2022
Merged

Add leading zeros to the milliseconds part of a log timestamp#13
smirnoal merged 1 commit into
aws:mainfrom
smirnoal:main

Conversation

@smirnoal

Copy link
Copy Markdown
Contributor

Description of changes:
Milliseconds part of a log timestamp now padded with leading zeroes

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@keshayadkeshayad left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

logger_handler.setFormatter(
logging.Formatter(
"[%(levelname)s]\t%(asctime)s.%(msecs)dZ\t%(aws_request_id)s\t%(message)s\n",
"[%(levelname)s]\t%(asctime)s.%(msecs)03dZ\t%(aws_request_id)s\t%(message)s\n",

@carlzoghcarlzoghJan 13, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@carlzoghcarlzogh added enhancement New feature or request semver-patch Requires a patch semantic versioning bump labels May 26, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementNew feature or requestsemver-patchRequires a patch semantic versioning bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@smirnoal@carlzogh@keshayad