discovery: Fix cancel path in EKS audit log fetcher

camscale · camscale · commit 9cc258fe594f · 2025-10-28T19:26:16.000+11:00
Change a `break` to a `continue` when a context is done so we do not
continue to send an empty payload on cleanup.
diff --git a/lib/srv/discovery/eks_audit_log_fetcher.go b/lib/srv/discovery/eks_audit_log_fetcher.go
@@ -69,10 +69,9 @@ func (f *eksAuditLogFetcher) Run(ctx context.Context) error {
 		if len(events) == 0 {
 			select {
 			case <-ctx.Done():
-				break
 			case <-time.After(logPollInterval):
-				continue
 			}
+			continue
 		}
 
 		if err := f.sendTAGKubeAuditLogEvents(ctx, events, cursor); err != nil {

Original file line number	Diff line number	Diff line change
`@@ -69,10 +69,9 @@ func (f *eksAuditLogFetcher) Run(ctx context.Context) error {`
`69`	`69`	`if len(events) == 0 {`
`70`	`70`	`select {`
`71`	`71`	`case <-ctx.Done():`
`72`		`- break`
`73`	`72`	`case <-time.After(logPollInterval):`
`74`		`- continue`
`75`	`73`	`}`
	`74`	`+ continue`
`76`	`75`	`}`
`77`	`76`
`78`	`77`	`if err := f.sendTAGKubeAuditLogEvents(ctx, events, cursor); err != nil {`