Skip to content

Commit

Permalink
Merge pull request #46 from Comcast/fix/no-need-to-close
Browse files Browse the repository at this point in the history
Remove un-needed closures.
  • Loading branch information
njharter authored Aug 22, 2017
2 parents 82d036f + c7449d4 commit 4f7ab9d
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions src/caduceus/http.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ type ServerHandler struct {
}

func (sh *ServerHandler) ServeHTTP(response http.ResponseWriter, request *http.Request) {
defer request.Body.Close()

sh.Info("Receiving incoming request...")

Expand Down Expand Up @@ -108,7 +107,6 @@ type ProfileHandler struct {
// ServeHTTP method of ProfileHandler will output the most recent messages
// that the main handler has successfully dealt with
func (ph *ProfileHandler) ServeHTTP(response http.ResponseWriter, request *http.Request) {
defer request.Body.Close()

ph.Info("Receiving request for server stats...")
stats := ph.profilerData.Report()
Expand Down

0 comments on commit 4f7ab9d

Please sign in to comment.