X-Git-Url: https://git.giacomolli.uk/onechat/blobdiff_plain/8d9547f6e7ffa1e95ad25d9f8f2e7cfd23d992d0..195c3fbb075383ac47bba310eb701b02774e8d10:/whatsapp_groups.go

diff --git a/whatsapp_groups.go b/whatsapp_groups.go
index e8111c3..6e9bed3 100644
--- a/whatsapp_groups.go
+++ b/whatsapp_groups.go
@@ -2,13 +2,17 @@ package main
 
 import (
 	"context"
-	"log"
+	"log/slog"
 	"maps"
 	"sync"
 	"time"
 )
 
 const (
+	// whatsappGroupLookupTimeout is the longest time of a read of group
+	// names.
+	whatsappGroupLookupTimeout = 10 * time.Second
+
 	// whatsappGroupsRefresh is the shortest time between two reads of the
 	// groups from WhatsApp for a group with no name.
 	whatsappGroupsRefresh = time.Minute
@@ -23,7 +27,7 @@ const (
 
 // newWhatsAppGroups returns the names of the groups with no names yet. logger
 // gets the failures of a read.
-func newWhatsAppGroups(logger *log.Logger) *WhatsAppGroups {
+func newWhatsAppGroups(logger *slog.Logger) *WhatsAppGroups {
 	return &WhatsAppGroups{
 		logger: logger,
 		names:  make(map[string]string),
@@ -36,7 +40,7 @@ func newWhatsAppGroups(logger *log.Logger) *WhatsAppGroups {
 // read, and readAt the time of the last read that succeeded. The mutex
 // protects all fields after logger.
 type WhatsAppGroups struct {
-	logger *log.Logger
+	logger *slog.Logger
 
 	mutex   sync.Mutex
 	names   map[string]string
@@ -49,6 +53,7 @@ type WhatsAppGroups struct {
 // again, at most once in whatsappGroupsRefresh.
 func (g *WhatsAppGroups) name(ctx context.Context, link WhatsAppLink, id string) string {
 	g.mutex.Lock()
+
 	name, found := g.names[id]
 	fresh := time.Since(g.triedAt) < whatsappGroupsRefresh
 	g.mutex.Unlock()
@@ -57,6 +62,7 @@ func (g *WhatsAppGroups) name(ctx context.Context, link WhatsAppLink, id string)
 		g.read(ctx, link)
 
 		g.mutex.Lock()
+
 		name, found = g.names[id]
 		g.mutex.Unlock()
 	}
@@ -75,12 +81,12 @@ func (g *WhatsAppGroups) read(ctx context.Context, link WhatsAppLink) bool {
 		return false
 	}
 
-	ctx, cancel := context.WithTimeout(ctx, whatsappLookupTime)
+	ctx, cancel := context.WithTimeout(ctx, whatsappGroupLookupTimeout)
 	defer cancel()
 
 	groups, err := link.client.GetJoinedGroups(ctx)
 	if err != nil {
-		g.logger.Printf("group names can be missing, because the list of groups did not load: %s", whatsappRedact(link.own, err.Error()))
+		g.logger.Warn("group list failed", "err", whatsappRedact(link.own, err.Error()))
 	}
 
 	g.mutex.Lock()