Start of error reporting

notification_log.go

@@ -9,19 +9,22 @@ import (
 	"smon/notification"
 
 	// Standard
+	"database/sql"
+	"encoding/json"
 	"time"
 )
 
 type NotificationSend struct {
-	Prio         int
+	Prio          int
-	Service      string
+	Service       string
-	ID           int
+	ID            int
-	UUID         string
+	UUID          string
-	Sent         time.Time `db:"send"`
+	Sent          time.Time `db:"send"`
-	OK           bool
+	OK            bool
-	Error        []byte
+	Error         sql.NullString
-	Acknowledged bool
+	ErrorIndented string
-	TriggerName  string `db:"trigger_name"`
+	Acknowledged  bool
+	TriggerName   string `db:"trigger_name"`
 }
 
 func notificationLog(notificationService *notification.Service, problemID int, err error) {
@@ -51,7 +54,7 @@ func notificationsSent() (nss []NotificationSend, err error) {
 			ns.uuid,
 			ns.send,
 			ns.ok,
-			ns.error,
+			ns.error::varchar,
 			ns.acknowledged
 			
 		FROM
@@ -76,6 +79,15 @@ func notificationsSent() (nss []NotificationSend, err error) {
 			err = werr.Wrap(err)
 			return
 		}
+
+		// Error contains json (can be NULL),
+		// and can at least be presented indented.
+		foo := make(map[string]any)
+		json.Unmarshal([]byte(ns.Error.String), &foo)
+		var j []byte
+		j, err = json.MarshalIndent(foo, "", "    ")
+		ns.ErrorIndented = string(j)
+
 		nss = append(nss, ns)
 	}
 	return

static/css/default_light/notifications.css

@@ -1,6 +1,6 @@
 #notifications {
   display: grid;
-  grid-template-columns: repeat(4, min-content);
+  grid-template-columns: repeat(5, min-content);
   grid-gap: 4px 16px;
   margin-top: 32px;
   margin-bottom: 32px;

static/css/default_light/problems.css

@@ -46,7 +46,7 @@
 }
 #areas .area .section .name {
   color: #000;
-  grid-column: -1;
+  grid-column: 1 / -1;
   font-weight: bold !important;
   line-height: 24px;
 }

static/css/gruvbox/notifications.css

@@ -1,6 +1,6 @@
 #notifications {
   display: grid;
-  grid-template-columns: repeat(4, min-content);
+  grid-template-columns: repeat(5, min-content);
   grid-gap: 4px 16px;
   margin-top: 32px;
   margin-bottom: 32px;

static/css/gruvbox/problems.css

@@ -46,7 +46,7 @@
 }
 #areas .area .section .name {
   color: #f7edd7;
-  grid-column: -1;
+  grid-column: 1 / -1;
   font-weight: bold !important;
   line-height: 24px;
 }

static/less/notifications.less

@@ -2,7 +2,7 @@
 
 #notifications {
 	display: grid;
-	grid-template-columns: repeat(4, min-content);
+	grid-template-columns: repeat(5, min-content);
 	grid-gap: 4px 16px;
 	margin-top: 32px;
 	margin-bottom: 32px;

static/less/problems.less

@@ -52,7 +52,7 @@
 
 			.name {
 				color: @text2;
-				grid-column: 1 / -1;
+				grid-column: ~"1 / -1";
 				font-weight: bold !important;
 				line-height: 24px;
 			}

views/pages/notifications.gotmpl

@@ -10,11 +10,13 @@
 		<div class="header">OK</div>
 		<div class="header">Trigger name</div>
 		<div class="header">Service</div>
+		<div class="header">Error</div>
 		{{ range .Data.Notifications }}
 		<div>{{ format_time .Sent }}</div>
 		<div>{{ if .OK }}✔{{ else }}✗{{ end }}</div>
 		<div>{{ .TriggerName }}</div>
 		<div>{{ .Prio }}:{{ .Service }}</div>
+	<div><pre>{{ if .Error.Valid }}{{ .ErrorIndented }}{{ end }}</pre></div>
 		{{ end }}
 	</div>