Merge pull request #1958 from weaveworks/mike/middleware/errorhandler

middleware/errorhandler: Fix a bug which meant it never works
weaveworks · Oct 31, 2016 · 96058b1 · 96058b1
2 parents 654ba35 + 6173e54
commit 96058b1
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/common/middleware/errorhandler.go b/common/middleware/errorhandler.go
@@ -49,12 +49,12 @@ func newErrorInterceptor(w http.ResponseWriter, code int) *errorInterceptor {
 }
 
 // Header implements http.ResponseWriter
-func (i errorInterceptor) Header() http.Header {
+func (i *errorInterceptor) Header() http.Header {
 	return i.headers
 }
 
 // WriteHeader implements http.ResponseWriter
-func (i errorInterceptor) WriteHeader(code int) {
+func (i *errorInterceptor) WriteHeader(code int) {
 	if i.gotCode {
 		panic("errorInterceptor.WriteHeader() called twice")
 	}
@@ -69,7 +69,7 @@ func (i errorInterceptor) WriteHeader(code int) {
 }
 
 // Write implements http.ResponseWriter
-func (i errorInterceptor) Write(data []byte) (int, error) {
+func (i *errorInterceptor) Write(data []byte) (int, error) {
 	if !i.gotCode {
 		i.WriteHeader(http.StatusOK)
 	}