luxolus
/
open-consul
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

Ensure we write json header before writing bytes 

In net/http once we've issued a Write() the response is sent
over the wire including the header! The tests didn't catch
this because I used a net/http/httptest.RequestRecorder
which doesn't follow those semantics.
This commit is contained in:
Jacques Fuentes 2014-04-17 14:53:07 -04:00
parent 236a8416f8
commit 852a47fe37
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
command/agent/http.go
View File

@ -124,8 +124,8 @@ func (s *HTTPServer) wrap(handler func(resp http.ResponseWriter, req *http.Reque
if err = enc.Encode(obj); err != nil { if err = enc.Encode(obj); err != nil {
goto HAS_ERR goto HAS_ERR
} }
resp.Write(buf.Bytes())
resp.Header().Set("Content-Type", "application/json") resp.Header().Set("Content-Type", "application/json")
resp.Write(buf.Bytes())
} }
} }
return f return f