feat: Check not retryable status codes

Signed-off-by: Maksim Paskal <[email protected]>

pkg/webhook/webhook.go

@@ -15,6 +15,8 @@ package webhook
 import (
 	"bytes"
 	"context"
+	"fmt"
+	"net/http"
 	"os"
 
 	"github.com/hashicorp/go-retryablehttp"
@@ -26,6 +28,8 @@ import (
 
 var client = &retryablehttp.Client{}
 
+var errHTTPNotOK = errors.New("http result not OK")
+
 func SetHTTPClient(c *retryablehttp.Client) {
 	client = c
 }
@@ -80,5 +84,11 @@ func SendWebHook(ctx context.Context, obj *template.MessageType) error {
 	}
 	defer resp.Body.Close()
 
+	log.Infof("response status: %s", resp.Status)
+
+	if resp.StatusCode != http.StatusOK {
+		return errors.Wrap(errHTTPNotOK, fmt.Sprintf("StatusCode=%d", resp.StatusCode))
+	}
+
 	return nil
 }

pkg/webhook/webhook_test.go

@@ -34,6 +34,12 @@ import (
 var retryableRequstCount = 0
 
 var ts = httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+	if r.RequestURI == "/-/400" {
+		w.WriteHeader(http.StatusBadRequest)
+
+		return
+	}
+
 	if r.RequestURI == "/test-retryable" {
 		retryableRequstCount++
 
@@ -121,6 +127,15 @@ func TestWebHook(t *testing.T) { //nolint:funlen,tparallel
 			},
 			HTTPClient: retryClientDefault,
 		},
+		{
+			Name: "TestRetryableCustomStatusCodes",
+			Args: map[string]string{
+				"webhook.url": ts.URL + "/-/400",
+			},
+			HTTPClient:   retryClientDefault,
+			Error:        true,
+			ErrorMessage: "http result not OK",
+		},
 		{
 			Name: "ValidHookAndTemplate",
 			Args: map[string]string{