use pointer struct, remove goroutine

Author: LumePart

src/logging/logging.go

@@ -7,7 +7,7 @@ import (
 )
 
 
-func Init(level string, notifyClient NotificationClient) {
+func Init(level string, notifyClient *NotificationClient) {
 	baseHandler := slog.NewTextHandler( os.Stdout, &slog.HandlerOptions{
 		Level: getLogLevel(level),
 		AddSource: true,

src/logging/notify.go

@@ -23,8 +23,8 @@ type NotificationClient struct {
 	Cfg config.NotifyConfig
 }
 
-func InitNotify(cfg config.NotifyConfig) NotificationClient {
-	return NotificationClient{
+func InitNotify(cfg config.NotifyConfig) *NotificationClient {
+	return &NotificationClient{
 		Cfg: cfg,
 	}
 }
@@ -123,7 +123,7 @@ func formatRecordJSON(n Notification) string {
 }
 
 
-func (c NotificationClient) SendNotification(n Notification) {
+func (c *NotificationClient) SendNotification(n Notification) {
 	var err error
 	switch c.Cfg.Service {
 		case "matrix":

src/logging/notify_handler.go

@@ -1,15 +1,15 @@
 package logging
 
 import (
-	"log/slog"
 	"context"
+	"log/slog"
 	"time"
 )
 // slog handler that checks whether to send notifications
 
 type notifyHandler struct {
 	handler   slog.Handler
-	notify NotificationClient
+	notify *NotificationClient
 }
 
 type Notification struct {
@@ -28,7 +28,7 @@ func (h *notifyHandler) Handle(ctx context.Context, r slog.Record) error {
 	if shouldNotify(r) {
 		// send notification in another goroutine
 		notifyStruct := recordToStruct(r)
-		go h.notify.SendNotification(notifyStruct) 
+		h.notify.SendNotification(notifyStruct)
 	}
 	return h.handler.Handle(ctx, r)
 }