Only delete notification subs for HTTP 410

diff --git a/src/littlesongplace/push_notifications.py b/src/littlesongplace/push_notifications.py
index dd4aa735230208038b28680392d2bd46a20f5ca4..53cf1346b46a3b56ba29d370b6330fbd7c754a3b 100644 (file)
--- a/src/littlesongplace/push_notifications.py
+++ b/src/littlesongplace/push_notifications.py
@@ -199,9 +199,12 @@ def _do_push(app, userids, title, body, url, setting):
                     sent_notifications += 1
                 except pywebpush.WebPushException as ex:
                     # Failed to send notification, delete this subscription
-                    app.logger.warning(f"Deleting dead push subscription: {subid} - {ex}")
-                    db.query("DELETE FROM users_push_subscriptions WHERE subid = ?", [subid])
-                    db.commit()
+                    if (ex.response is not None) and (ex.response.status_code == 410):
+                        app.logger.warning(f"Deleting dead push subscription: {subid} - {ex}")
+                        db.query("DELETE FROM users_push_subscriptions WHERE subid = ?", [subid])
+                        db.commit()
+                    else:
+                        app.logger.warning(f"Failed to send push subscription: {subid} - {ex}")
 
         if sent_notifications > 0:
             app.logger.info(f"Pushed {sent_notifications} notifications")