Skip summary for slack alerts too

cloudprober · Oct 17, 2023 · 4905408 · 4905408
1 parent f4bef47
commit 4905408
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/probes/alerting/notifier/slack/slack.go b/probes/alerting/notifier/slack/slack.go
@@ -6,6 +6,7 @@ import (
 	"context"
 	"encoding/json"
 	"fmt"
+	"io"
 	"net/http"
 	"os"
 
@@ -97,7 +98,8 @@ func (c *Client) Notify(ctx context.Context, alertFields map[string]string) erro
 
 	// check status code, return error if not 200
 	if resp.StatusCode != http.StatusOK {
-		return fmt.Errorf("Slack webhook returned status code %d", resp.StatusCode)
+		b, _ := io.ReadAll(resp.Body)
+		return fmt.Errorf("slack webhook returned error; statusCode: %d, response: %s", resp.StatusCode, string(b))
 	}
 
 	return nil
@@ -106,6 +108,6 @@ func (c *Client) Notify(ctx context.Context, alertFields map[string]string) erro
 // createMessage creates a new Slack webhook message, from the alertFields
 func createMessage(alertFields map[string]string) webhookMessage {
 	return webhookMessage{
-		Text: alertFields["details"] + "\n\nDetails:\n" + alertinfo.FieldsToString(alertFields, "details"),
+		Text: alertFields["details"] + "\n\nDetails:\n" + alertinfo.FieldsToString(alertFields, "details", "summary"),
 	}
 }