[GitHub] incubator-trafficcontrol issue #642: Add Traffic Ops username to access log

2017-06-06 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/642 this is what the log line looks like now: 127.0.0.1 - - [05/Jun/2017:13:54:34 -0600] "GET /api/1.2/logs/newcount HTTP/1.1" 200 51 25823 "Mozilla/5.0 (Macintosh; Intel

[GitHub] incubator-trafficcontrol issue #642: Add Traffic Ops username to access log

2017-06-05 Thread rob05c
Github user rob05c commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/642 This PR doesn't modify the log format, which was already logging `%u` (which apparently broke sometime in the past), it only makes the logger aware of the username. --- If your

[GitHub] incubator-trafficcontrol issue #642: Add Traffic Ops username to access log

2017-06-05 Thread dewrich
Github user dewrich commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/642 I think putting the username at the "end" of the line is better in case there are any logs scrapers concerned with that format. Otherwise I think the localUsername vs. the

[GitHub] incubator-trafficcontrol issue #642: Add Traffic Ops username to access log

2017-06-05 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/642 Tested locally, here were my results: 127.0.0.1 - localUsername [05/Jun/2017:13:38:48 -0600] "GET /api/1.2/logs/newcount HTTP/1.1" 200 51 7370 "Mozilla/5.0 (Macintosh;