feat(monitor): force non-empty error messages for Uptime Kuma (#624)

favonia · Oct 24, 2023 · a9bce5c · a9bce5c
1 parent 1832760
commit a9bce5c
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/internal/monitor/updatekuma.go → internal/monitor/uptimekuma.go b/internal/monitor/updatekuma.go → internal/monitor/uptimekuma.go
@@ -156,17 +156,22 @@ func (h *UptimeKuma) Success(ctx context.Context, ppfmt pp.PP, _message string)
 }
 
 // Start does nothing.
-func (h *UptimeKuma) Start(ctx context.Context, ppfmt pp.PP, message string) bool {
+func (h *UptimeKuma) Start(ctx context.Context, ppfmt pp.PP, _message string) bool {
 	return true
 }
 
 // Failure pings the server with status=down.
 func (h *UptimeKuma) Failure(ctx context.Context, ppfmt pp.PP, message string) bool {
+	if len(message) == 0 {
+		// Uptime Kuma seems to keep the previous message (even if it was for success) by default.
+		// We thus send an explicit message to overwrite it.
+		message = "Failing"
+	}
 	return h.ping(ctx, ppfmt, UptimeKumaRequest{Status: "down", Msg: message, Ping: ""})
 }
 
 // Log does nothing.
-func (h *UptimeKuma) Log(ctx context.Context, ppfmt pp.PP, message string) bool {
+func (h *UptimeKuma) Log(ctx context.Context, ppfmt pp.PP, _message string) bool {
 	return true
 }
 

diff --git a/internal/monitor/updatekuma_test.go → internal/monitor/uptimekuma_test.go b/internal/monitor/updatekuma_test.go → internal/monitor/uptimekuma_test.go
@@ -195,6 +195,21 @@ func TestUptimeKumaEndPoints(t *testing.T) {
 				)
 			},
 		},
+		"failure/empty": {
+			func(ppfmt pp.PP, m monitor.Monitor) bool {
+				return m.Failure(context.Background(), ppfmt, "")
+			},
+			"/", "down", "Failing", "",
+			[]action{ActionOk},
+			ActionAbort, true,
+			true,
+			func(m *mocks.MockPP) {
+				gomock.InOrder(
+					m.EXPECT().Warningf(pp.EmojiUserWarning, "The Uptime Kuma URL (redacted) uses HTTP; please consider using HTTPS"),
+					m.EXPECT().Infof(pp.EmojiNotification, "Successfully pinged Uptime Kuma"),
+				)
+			},
+		},
 		"log": {
 			func(ppfmt pp.PP, m monitor.Monitor) bool {
 				return m.Log(context.Background(), ppfmt, "message")