fix formatting of latency in combined access log (!49) · Merge requests · GitLab.org / labkit

Igor requested to merge access-log-combined-latency into master Mar 05, 2020

I noticed while running gdk locally that the latency field in the combined access log has the wrong type:

2020-03-05_09:30:41.77301 gitlab-workhorse      : localhost:3000 127.0.0.1 - - [2020/03/05:10:30:41 +0100] "GET /favicon.ico HTTP/1.1" 502 583 "http://localhost:3000/root/example/-/tree/master" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.122 Safari/537.36" %!f(int64=001)

See the %!f(int64=001) at the end.

This patch changes the formatting for latency to be %d for int64.

Edited Mar 12, 2020 by Igor

fix formatting of latency in combined access log

Merge request reports