BUG/MINOR: adapters: close writter when new apache middleware is created

Author: rmaticevic

adapters/adapters.go

@@ -17,6 +17,7 @@ package adapters
 
 import (
 	"fmt"
+	"io"
 	"math/rand"
 	"net/http"
 	"runtime"
@@ -30,6 +31,10 @@ import (
 	"github.com/sirupsen/logrus"
 )
 
+var (
+	apacheLogWritter *io.PipeWriter
+)
+
 // Adapter is just a wrapper over http handler function
 type Adapter func(http.Handler) http.Handler
 
@@ -176,7 +181,11 @@ func UniqueIDMiddleware(entry *logrus.Entry) Adapter {
 
 func ApacheLogMiddleware(entry *logrus.Entry, format *apachelog.ApacheLog) Adapter {
 	return func(h http.Handler) http.Handler {
-		return format.Wrap(h, entry.Logger.Writer())
+		if apacheLogWritter != nil {
+			apacheLogWritter.Close()
+		}
+		apacheLogWritter = entry.Logger.Writer()
+		return format.Wrap(h, apacheLogWritter)
 	}
 }