gsadmin/pulse
1
0
Fork 0
mirror of https://github.com/rcourtman/Pulse.git synced 2026-09-21 10:43:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: ensure Gotify webhook links use full Pulse URL instead of relative path (addresses #415)

Browse Source
This commit is contained in:
Pulse Monitor
2025-09-04 17:48:00 +00:00
parent 2fe5117936
commit 21b168d287
1 changed files with 8 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/notifications/notifications.go
+8 -10
View File
@@ -823,16 +823,14 @@ func (n *NotificationManager) prepareWebhookData(alert *alerts.Alert, customFiel
duration := time.Since(alert.StartTime)
// Construct full Pulse URL if publicURL is configured
instance := alert.Instance
if n.publicURL != "" && alert.Instance != "" {
// If alert.Instance is just the instance name (e.g., "pve-10.0.1.21")
// construct the full URL to view this instance in Pulse
if !strings.HasPrefix(alert.Instance, "http://") && !strings.HasPrefix(alert.Instance, "https://") {
// Remove trailing slash from publicURL if present
publicURL := strings.TrimRight(n.publicURL, "/")
// Construct the full URL with the instance path
instance = fmt.Sprintf("%s/%s", publicURL, alert.Instance)
}
// The Instance field should contain the full URL to the Pulse dashboard
instance := ""
if n.publicURL != "" {
// Remove trailing slash from publicURL if present
instance = strings.TrimRight(n.publicURL, "/")
} else if alert.Instance != "" && (strings.HasPrefix(alert.Instance, "http://") || strings.HasPrefix(alert.Instance, "https://")) {
// If publicURL is not set but alert.Instance contains a full URL, use it
instance = alert.Instance
}
return WebhookPayloadData{