fix: Drop 0 TTL WebPush notifications if they aren't delivered the fi…

…rst time (#210)

* Drop 0 TTL WebPush notifications if they aren't delivered the first time

This follows the logic used in Python autoendpoint.

* Update tests

autoendpoint/src/routers/webpush.rs

@@ -71,6 +71,14 @@ impl Router for WebPushRouter {
             }
         }
 
+        if notification.headers.ttl == 0 {
+            trace!(
+                "Notification has a TTL of zero and was not successfully \
+                 delivered, dropping it"
+            );
+            return Ok(self.make_delivered_response(notification));
+        }
+
         // Save notification, node is not present or busy
         trace!("Node is not present or busy, storing notification");
         self.store_notification(notification).await?;

tests/test_integration_all_rust.py

@@ -764,7 +764,7 @@ def test_ttl_0_not_connected(self):
         data = str(uuid.uuid4())
         client = yield self.quick_register()
         yield client.disconnect()
-        yield client.send_notification(data=data, ttl=0)
+        yield client.send_notification(data=data, ttl=0, status=201)
         yield client.connect()
         yield client.hello()
         result = yield client.get_notification(timeout=0.5)