fix(monitor): check if LastSendNotification is zero (#23782)

zexi · web-flow · commit ce8f84add6c7 · 2025-11-20T18:04:10.000+08:00
diff --git a/pkg/monitor/models/notification.go b/pkg/monitor/models/notification.go
@@ -237,7 +237,11 @@ func (n *SNotification) ShouldSendNotification() bool {
 	if n.Frequency == 0 {
 		return true
 	}
-	if int64(time.Now().Sub(n.LastSendNotification)/time.Second)+int64(60) >= n.Frequency {
+	// 如果从未发送过通知（LastSendNotification 为零值），允许第一次发送
+	if n.LastSendNotification.IsZero() {
+		return true
+	}
+	if int64(time.Since(n.LastSendNotification)/time.Second)+int64(60) >= n.Frequency {
 		return true
 	}
 	return false

Original file line number	Diff line number	Diff line change
`@@ -237,7 +237,11 @@ func (n *SNotification) ShouldSendNotification() bool {`
`237`	`237`	`if n.Frequency == 0 {`
`238`	`238`	`return true`
`239`	`239`	`}`
`240`		`- if int64(time.Now().Sub(n.LastSendNotification)/time.Second)+int64(60) >= n.Frequency {`
	`240`	`+ // 如果从未发送过通知（LastSendNotification 为零值），允许第一次发送`
	`241`	`+ if n.LastSendNotification.IsZero() {`
	`242`	`+ return true`
	`243`	`+ }`
	`244`	`+ if int64(time.Since(n.LastSendNotification)/time.Second)+int64(60) >= n.Frequency {`
`241`	`245`	`return true`
`242`	`246`	`}`
`243`	`247`	`return false`