BUGFIX: Panic when enabling/disabling monitors 🪲

benc-uk · Apr 2, 2023 · bfe8b6d · bfe8b6d
1 parent b2b2ce0
commit bfe8b6d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/services/runner/monitor/http.go b/services/runner/monitor/http.go
@@ -96,7 +96,7 @@ func (m *Monitor) runHTTP() (*types.Result, map[string]any) {
 		}
 
 		match := re.FindStringSubmatch(bodyStr)
-		if match != nil && len(match) > 1 {
+		if len(match) > 1 {
 			regexMatch = match[1]
 		}
 	}

diff --git a/services/runner/monitor/monitor.go b/services/runner/monitor/monitor.go
@@ -151,5 +151,8 @@ func (m *Monitor) run() (bool, *types.Result) {
 
 func (m *Monitor) Stop() {
 	log.Println("### Stopping monitor", m.Name)
-	m.ticker.Stop()
+
+	if m.ticker != nil {
+		m.ticker.Stop()
+	}
 }