Merge pull request #259 from xmidt-org/feature/log-security-requested…

…-items

added user-agent and the sallust default fields to the request logger

xhttp/xhttpserver/server.go

@@ -53,12 +53,17 @@ type Options struct {
 // NewServerChain produces the standard constructor chain for a server, primarily using configuration.
 func NewServerChain(o Options, l *zap.Logger, fbs ...sallusthttp.FieldBuilder) alice.Chain {
 	bs := sallusthttp.Builders{}
+	bs.AddFields(fbs...)
+	bs.Add(sallusthttp.DefaultFields)
+	bs.Add(func(r *http.Request, l *zap.Logger) *zap.Logger {
+		return l.With(zap.String("userAgent", r.UserAgent()))
+	})
+
 	chain := alice.New(
 		ResponseHeaders{Header: o.Header}.Then,
 		Busy{MaxConcurrentRequests: o.MaxConcurrentRequests}.Then,
 	)
 
-	bs.AddFields(fbs...)
 	if !o.DisableTracking {
 		chain = chain.Append(UseTrackingWriter)
 	}