Update library and add more comments.

drkaka · Mar 23, 2017 · 23d693f · 23d693f
1 parent cf84085
commit 23d693f
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/glide.lock b/glide.lock
diff --git a/glide.yaml b/glide.yaml
@@ -3,7 +3,7 @@ import:
 - package: github.com/satori/go.uuid
   version: ^1.1.0
 - package: go.uber.org/zap
-  version: ^1.0.0-rc.2
+  version: ^1.0.0
   subpackages:
   - zapcore
 testImport:

diff --git a/lg.go b/lg.go
@@ -60,9 +60,11 @@ func L(ctx context.Context) *zap.Logger {
 // LogRequest to log every request.
 func LogRequest(next http.Handler) http.Handler {
 	fn := func(w http.ResponseWriter, r *http.Request) {
+		// Add a requestID field to logger
 		l := log.With(zap.String("requestID", uuid.NewV1().String()))
-		ctx := r.Context()
-		ctx = context.WithValue(ctx, requestIDKey, l)
+		// Add logger to context
+		ctx := context.WithValue(r.Context(), requestIDKey, l)
+		// Request with this new context.
 		r = r.WithContext(ctx)
 
 		// Start timer