gvsafronov/tile38
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Allow 201 & 202 status code on webhooks

Browse Source
This commit is contained in:
Steven Wolfe 2019-02-05 14:49:01 -07:00
parent bfa35d5db9
commit 628e0882b8
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
internal/endpoint/http.go
View File

@ -57,8 +57,10 @@ func (conn *HTTPConn) Send(msg string) error {
if _, err := io.Copy(ioutil.Discard, resp.Body); err != nil { if _, err := io.Copy(ioutil.Discard, resp.Body); err != nil {
return err return err
} }
// we only care about the 200 response // Only allow responses with status code 200, 201, and 202
if resp.StatusCode != 200 { if resp.StatusCode != http.StatusOK &&
resp.StatusCode != http.StatusCreated &&
resp.StatusCode != http.StatusAccepted {
return fmt.Errorf("invalid status: %s", resp.Status) return fmt.Errorf("invalid status: %s", resp.Status)
} }
return nil return nil