Skip to content
Snippets Groups Projects

fix formatting of latency in combined access log

Merged fix formatting of latency in combined access log
access-log-combined-latency into master
1 unresolved thread
Merged Igor requested to merge access-log-combined-latency into master
1 unresolved thread

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 by Igor

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply