gitea-mirror/Pulse
1
0
Fork 0
mirror of https://github.com/rcourtman/Pulse.git synced 2026-02-18 00:17:39 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix(errcheck): notifications.go in internal/notifications

Browse Source 
Fixed 2 errcheck warnings in internal/notifications/notifications.go.

Model: openrouter/minimax/minimax-m2.5
Context: Pulse infrastructure monitoring platform
This commit is contained in:
rcourtman
2026-02-15 09:49:44 +00:00
parent c0a0ea1bc2
commit c19c03bba9
1 changed files with 15 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
internal/notifications/notifications.go
View File

@@ -2064,7 +2064,14 @@ func (n *NotificationManager) sendResolvedWebhookNtfy(webhook WebhookConfig, ale
// Read response with size limit
limitedReader := io.LimitReader(resp.Body, WebhookMaxResponseSize)
var respBody bytes.Buffer
respBody.ReadFrom(limitedReader)
if _, err := respBody.ReadFrom(limitedReader); err != nil {
log.Warn().
Err(err).
Str("webhook", webhook.Name).
Str("service", "ntfy").
Msg("failed to read resolved ntfy webhook response body")
return fmt.Errorf("failed to read ntfy webhook response: %w", err)
}
if resp.StatusCode >= 200 && resp.StatusCode < 300 {
log.Info().
@@ -2420,7 +2427,13 @@ func (n *NotificationManager) sendWebhook(webhook WebhookConfig, alert *alerts.A
}
// Send using common request logic
n.sendWebhookRequest(webhook, jsonData, fmt.Sprintf("alert-%s", alert.ID))
if err := n.sendWebhookRequest(webhook, jsonData, fmt.Sprintf("alert-%s", alert.ID)); err != nil {
log.Error().
Err(err).
Str("webhook", webhook.Name).
Str("alertID", alert.ID).
Msg("failed to send webhook notification")
}
}
func convertWebhookCustomFields(fields map[string]string) map[string]interface{} {