fix(slack): did not check JSON marshaling errors

containrrr · Nov 5, 2023 · 481b5ab · 481b5ab
1 parent 26fdc62
commit 481b5ab
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/pkg/services/slack/slack.go b/pkg/services/slack/slack.go
@@ -84,9 +84,10 @@ func (service *Service) sendAPI(config *Config, payload interface{}) error {
 
 func (service *Service) sendWebhook(config *Config, payload interface{}) error {
 	payloadBytes, err := json.Marshal(payload)
-	var res *http.Response
-	res, err = http.Post(config.Token.WebhookURL(), jsonclient.ContentType, bytes.NewBuffer(payloadBytes))
-
+	if err != nil {
+		return fmt.Errorf("failed to marshal payload: %w", err)
+	}
+	res, err := http.Post(config.Token.WebhookURL(), jsonclient.ContentType, bytes.NewBuffer(payloadBytes))
 	if err != nil {
 		return fmt.Errorf("failed to invoke webhook: %w", err)
 	}