Browse Source

Change case of alert json to initial lower letter.

This is more in line with common practice,
and with the webhook in alertmanager this will
be more directly exposed to users.
pull/751/head
Brian Brazil 10 years ago
parent
commit
75e8b48c87
  1. 16
      notification/notification.go
  2. 2
      notification/notification_test.go

16
notification/notification.go

@ -143,14 +143,14 @@ func (n *NotificationHandler) sendNotifications(reqs NotificationReqs) error {
alerts := make([]map[string]interface{}, 0, len(reqs)) alerts := make([]map[string]interface{}, 0, len(reqs))
for _, req := range reqs { for _, req := range reqs {
alerts = append(alerts, map[string]interface{}{ alerts = append(alerts, map[string]interface{}{
"Summary": req.Summary, "summary": req.Summary,
"Description": req.Description, "description": req.Description,
"Labels": req.Labels, "labels": req.Labels,
"Payload": map[string]interface{}{ "payload": map[string]interface{}{
"Value": req.Value, "value": req.Value,
"ActiveSince": req.ActiveSince, "activeSince": req.ActiveSince,
"GeneratorURL": req.GeneratorURL, "generatorURL": req.GeneratorURL,
"AlertingRule": req.RuleString, "alertingRule": req.RuleString,
}, },
}) })
} }

2
notification/notification_test.go

@ -81,7 +81,7 @@ func TestNotificationHandler(t *testing.T) {
// Correct message. // Correct message.
summary: "Summary", summary: "Summary",
description: "Description", description: "Description",
message: `[{"Description":"Description","Labels":{"instance":"testinstance"},"Payload":{"ActiveSince":"0001-01-01T00:00:00Z","AlertingRule":"Test rule string","GeneratorURL":"prometheus_url","Value":"0.3333333333333333"},"Summary":"Summary"}]`, message: `[{"description":"Description","labels":{"instance":"testinstance"},"payload":{"activeSince":"0001-01-01T00:00:00Z","alertingRule":"Test rule string","generatorURL":"prometheus_url","value":"0.3333333333333333"},"summary":"Summary"}]`,
}, },
} }

Loading…
Cancel
Save