Merge pull request #1197 from dtan4/nil-webhook-200

Return 200 even if non-action webhook is come
This commit is contained in:
Brad Rydzewski 2015-09-13 10:32:45 -07:00
commit 5d858ec30c

View file

@ -40,6 +40,11 @@ func PostHook(c web.C, w http.ResponseWriter, r *http.Request) {
return return
} }
if hook == nil {
w.WriteHeader(http.StatusOK)
return
}
// in some cases we have neither a hook nor error. An example // in some cases we have neither a hook nor error. An example
// would be GitHub sending a ping request to the URL, in which // would be GitHub sending a ping request to the URL, in which
// case we'll just exit quiely with an 'OK' // case we'll just exit quiely with an 'OK'