bug: silence missing TTL errors from sentry log

Missing TTL headers are being logged resulting in high levels of noise
in the Sentry logs. Set the "log_exception" param to False to prevent
the message from going to the system log.

closes #370

autopush/endpoint.py

@@ -246,7 +246,7 @@ def _router_fail_err(self, fail):
         fail.trap(RouterException)
         exc = fail.value
         if exc.log_exception:
-            log.err(fail, **self._client_info)
+            log.err(fail, **self._client_info)  # pragma nocover
         if 200 <= exc.status_code < 300:
             log.msg("Success", status_code=exc.status_code,
                     logged_status=exc.logged_status or "",

autopush/router/webpush.py

@@ -115,7 +115,8 @@ def _save_notification(self, uaid, notification, month_table):
                 "Missing TTL Header",
                 response_body="Missing TTL Header, see: %s" % TTL_URL,
                 status_code=400,
-                errno=111
+                errno=111,
+                log_exception=False,
             )
         if notification.ttl == 0:
             location = "%s/m/%s" % (self.ap_settings.endpoint_url,

autopush/tests/test_endpoint.py

@@ -317,7 +317,8 @@ def raise_error(*args):
                 "Missing TTL Header",
                 status_code=400,
                 response_body="Missing TTL Header",
-                errno=111
+                errno=111,
+                log_exception=False,
             )
 
         self.wp_router_mock.route_notification.side_effect = raise_error
@@ -1108,7 +1109,8 @@ def raise_error(*args):
                 "Provisioned throughput error",
                 status_code=503,
                 response_body="Retry Request",
-                errno=201
+                errno=201,
+                log_exception=False
             )
 
         self.wp_router_mock.route_notification.side_effect = raise_error