Use HTTP 400 instead of 500

kuebler · kuebler · commit e815160877a2 · 2022-09-07T16:05:33.000+02:00
diff --git a/internal/manager/receive.go b/internal/manager/receive.go
@@ -212,7 +212,7 @@ func (s *BitbucketWebhookReceiver) Handle(w http.ResponseWriter, r *http.Request
 	if err != nil {
 		msg := "Couldn't identify pipeline to run"
 		s.Logger.Errorf("%s: %s", msg, err)
-		http.Error(w, msg, http.StatusInternalServerError)
+		http.Error(w, msg, http.StatusBadRequest)
 		return
 	}
 	s.TriggeredPipelines <- cfg

Original file line number	Diff line number	Diff line change
`@@ -212,7 +212,7 @@ func (s BitbucketWebhookReceiver) Handle(w http.ResponseWriter, r http.Request`
`212`	`212`	`if err != nil {`
`213`	`213`	`msg := "Couldn't identify pipeline to run"`
`214`	`214`	`s.Logger.Errorf("%s: %s", msg, err)`
`215`		`- http.Error(w, msg, http.StatusInternalServerError)`
	`215`	`+ http.Error(w, msg, http.StatusBadRequest)`
`216`	`216`	`return`
`217`	`217`	`}`
`218`	`218`	`s.TriggeredPipelines <- cfg`