From 44f4918219f965f12e2046164c6154cd1bb5c1f1 Mon Sep 17 00:00:00 2001
From: Dinifarb <andreas.vogt89@bluewin.ch>
Date: Sun, 7 Jan 2024 13:24:51 +0100
Subject: [PATCH] fix(#1352): moving the stop signal after svc.Run()

Signed-off-by: Dinifarb <andreas.vogt89@bluewin.ch>
---
 pkg/initiate/initiate.go | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/pkg/initiate/initiate.go b/pkg/initiate/initiate.go
index 00162dd4..04b97561 100644
--- a/pkg/initiate/initiate.go
+++ b/pkg/initiate/initiate.go
@@ -13,9 +13,7 @@ const (
 	serviceName = "windows_exporter"
 )
 
-type windowsExporterService struct {
-	stopCh chan<- bool
-}
+type windowsExporterService struct{}
 
 var logger *eventlog.Log
 
@@ -39,7 +37,6 @@ loop:
 			}
 		}
 	}
-	s.stopCh <- true
 	return
 }
 
@@ -63,10 +60,13 @@ func init() {
 		}
 		_ = logger.Info(100, "Attempting to start exporter service")
 		go func() {
-			err = svc.Run(serviceName, &windowsExporterService{stopCh: StopCh})
+			err = svc.Run(serviceName, &windowsExporterService{})
 			if err != nil {
 				_ = logger.Error(102, fmt.Sprintf("Failed to start service: %v", err))
 			}
+			defer func() {
+				StopCh <- true
+			}()
 		}()
 	}
 }