fix(kcap): Disable alert senders in capture replay mode

rabbitstack · rabbitstack · commit dffe9b43a478 · 2024-09-11T22:25:39.000+02:00
If the capture is set in the configuration flags, either the capture is being generated or replayed. In a such scenario, alert senders are useless and should be disabled.
diff --git a/pkg/config/alertsender.go b/pkg/config/alertsender.go
@@ -35,8 +35,8 @@ var errAlertsenderConfig = func(sender string, err error) error {
 }
 
 func (c *Config) tryLoadAlertSenders() error {
-	if c.ForwardMode {
-		// In event forwarding mode, alert senders are useless
+	if c.ForwardMode || c.IsCaptureSet() {
+		// In event forwarding mode or capture control, alert senders are useless
 		return nil
 	}
 	configs := make([]alertsender.Config, 0)

Original file line number	Diff line number	Diff line change
`@@ -35,8 +35,8 @@ var errAlertsenderConfig = func(sender string, err error) error {`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`func (c *Config) tryLoadAlertSenders() error {`
`38`		`- if c.ForwardMode {`
`39`		`- // In event forwarding mode, alert senders are useless`
	`38`	`+ if c.ForwardMode \|\| c.IsCaptureSet() {`
	`39`	`+ // In event forwarding mode or capture control, alert senders are useless`
`40`	`40`	`return nil`
`41`	`41`	`}`
`42`	`42`	`configs := make([]alertsender.Config, 0)`