From 8ed94692c2072dc9673e446c3401bfcf49d62024 Mon Sep 17 00:00:00 2001 From: jeffyanta Date: Mon, 6 Jul 2026 10:40:18 -0400 Subject: [PATCH] Media is optional in message sender for consumers that don't require it --- messaging/sender.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/messaging/sender.go b/messaging/sender.go index f08c797..ba0eefb 100644 --- a/messaging/sender.go +++ b/messaging/sender.go @@ -135,8 +135,10 @@ func (s *Sender) Send( // media-free sends; the message is already persisted, so a resolution failure // just leaves Blob unset for clients to re-fetch. msgProto := msg.ToProto() - if err := hydrateMedia(ctx, s.media, []*messagingpb.Message{msgProto}); err != nil { - log.With(zap.Error(err)).Warn("Failure resolving media metadata") + if s.media != nil { + if err := hydrateMedia(ctx, s.media, []*messagingpb.Message{msgProto}); err != nil { + log.With(zap.Error(err)).Warn("Failure resolving media metadata") + } } // A retried send (same client message ID) already ran every side effect when