From f0302525a700d801864c41cfbbe016ba5cf75f97 Mon Sep 17 00:00:00 2001
From: Kendall Garner <17521368+kgarner7@users.noreply.github.com>
Date: Tue, 24 Dec 2024 22:35:19 +0000
Subject: [PATCH] fix(server): use cancellable context instead of Sleep for
 initial insights delay (#3593)

* bugfix(server): use cancellable contet instead of sleep for initial insights delay

* fix(server): initial delay time

Signed-off-by: Deluan <deluan@navidrome.org>

---------

Signed-off-by: Deluan <deluan@navidrome.org>
Co-authored-by: Deluan <deluan@navidrome.org>
---
 cmd/root.go | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/cmd/root.go b/cmd/root.go
index 7a3104543..3a1757be9 100644
--- a/cmd/root.go
+++ b/cmd/root.go
@@ -208,7 +208,11 @@ func startInsightsCollector(ctx context.Context) func() error {
 			return nil
 		}
 		log.Info(ctx, "Starting Insight Collector")
-		time.Sleep(conf.Server.DevInsightsInitialDelay)
+		select {
+		case <-time.After(conf.Server.DevInsightsInitialDelay):
+		case <-ctx.Done():
+			return nil
+		}
 		ic := CreateInsights()
 		ic.Run(ctx)
 		return nil