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

Make duration logging consistent #619

Merged
merged 1 commit into from Aug 15, 2016

Conversation

jangie
Copy link
Contributor

@jangie jangie commented Aug 15, 2016

GH #559 points out that the logging of the elapsed time is inconsistent
depending on the scale of the measured time; this is due to Duration’s
String handling.

With this PR, I propose that traefik logs millis, and not fractions of
millis.

GH traefik#559 points out that the logging of the elapsed time is inconsistent
depending on the scale of the measured time; this is due to Duration’s
String handling.

With this PR, I propose that traefik logs millis, and not fractions of
millis.
@jangie jangie force-pushed the consistent-duration-logging branch from ba84e36 to 3a736ad Compare August 15, 2016 12:43
@vdemeester
Copy link
Contributor

LGTM 🐵

@vdemeester vdemeester merged commit fb3bad3 into traefik:master Aug 15, 2016
@ldez ldez added kind/enhancement a new or improved feature. and removed status/2-needs-review labels Apr 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement a new or improved feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants