~emersion/soju: Bump update time on success in network.broadcastWebPush

a14ea80cb90b0aefbd2046a3f4c5ca0966f6aac1 — Simon Ser 4 days ago 93af53c

Bump update time on success in network.broadcastWebPush

Allows us to skip checks when the client sends WEBPUSH REGISTER
and avoid pruning subscriptions that still work fine.

patch browse .tar.gz

1 files changed, 5 insertions(+), 0 deletions(-)

M user.go

M user.go => user.go +5 -0

@@ 516,6 516,11 @@ func (net *network) broadcastWebPush(msg *irc.Message) {
 			}
 		} else if err != nil {
 			net.logger.Printf("failed to send Web push notification to endpoint %q: %v", sub.Endpoint, err)
+		} else {
+			// On success, bump the subscription's update time
+			if err := net.user.srv.db.StoreWebPushSubscription(ctx, net.user.ID, net.ID, &sub); err != nil {
+				net.logger.Printf("failed to store Web push subscription: %v", err)
+			}
 		}
 	}
 }