From 1e78e3e4b0c725157ee298698b737f2cf4718497 Mon Sep 17 00:00:00 2001 From: Simon Ser Date: Mon, 11 Jul 2022 22:54:51 +0200 Subject: [PATCH] dowstream: remove noop WEBPUSH REGISTER We still want to update the updated_at column when a client refreshes the subscription. --- downstream.go | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/downstream.go b/downstream.go index dac7ca0..6feadf0 100644 --- a/downstream.go +++ b/downstream.go @@ -3309,16 +3309,6 @@ func (dc *downstreamConn) handleMessageRegistered(ctx context.Context, msg *irc. } if oldSub != nil { - if oldSub.Keys.VAPID == newSub.Keys.VAPID && oldSub.Keys.Auth == newSub.Keys.Auth && oldSub.Keys.P256DH == newSub.Keys.P256DH { - // Nothing has changed, this is a no-op - dc.SendMessage(&irc.Message{ - Prefix: dc.srv.prefix(), - Command: "WEBPUSH", - Params: []string{"REGISTER", endpoint}, - }) - return nil - } - // Update the old subscription instead of creating a new one newSub.ID = oldSub.ID }