Skip to content

B
beego
Commit 24015e9a authored by astaxie's avatar astaxie Committed by GitHub
Browse files
Options

Merge pull request #2285 from cat2neat/fix-bodyclose 

Fix http body may not be closed
parents 27c59a80 9ff88f0b
Hide whitespace changes
Inline Side-by-side
Showing with 2 additions and 2 deletions
logs/jianliao.go
View file @ 24015e9a
......@@ -56,10 +56,10 @@ func (s *JLWriter) WriteMsg(when time.Time, msg string, level int) error {
if err != nil {
return err
}
defer resp.Body.Close()
if resp.StatusCode != http.StatusOK {
return fmt.Errorf("Post webhook failed %s %d", resp.Status, resp.StatusCode)
}
resp.Body.Close()
return nil
}
......
logs/slack.go
View file @ 24015e9a
......@@ -44,10 +44,10 @@ func (s *SLACKWriter) WriteMsg(when time.Time, msg string, level int) error {
if err != nil {
return err
}
defer resp.Body.Close()
if resp.StatusCode != http.StatusOK {
return fmt.Errorf("Post webhook failed %s %d", resp.Status, resp.StatusCode)
}
resp.Body.Close()
return nil
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment