Remove sentry exception capture for thumbnail errors

Author: AetherUnbound

api/api/utils/image_proxy/__init__.py

@@ -10,11 +10,9 @@
 
 import aiohttp
 import django_redis
-import sentry_sdk
 from aiohttp.client_exceptions import ClientResponseError
 from asgiref.sync import sync_to_async
 from redis.exceptions import ConnectionError
-from sentry_sdk import push_scope, set_context
 
 from api.utils.aiohttp import get_aiohttp_session
 from api.utils.asyncio import do_not_wait_for
@@ -192,28 +190,10 @@ async def get(
         exception_name = f"{exc.__class__.__module__}.{exc.__class__.__name__}"
         key = f"thumbnail_error:{exception_name}:{domain}:{month}"
         try:
-            count = await tallies_incr(key)
+            await tallies_incr(key)
         except ConnectionError:
             logger.warning("Redis connect failed, thumbnail errors not tallied.")
-            # We will use a counter to space out Sentry logs.
-            count = next(exception_iterator)
-
-        if count <= settings.THUMBNAIL_ERROR_INITIAL_ALERT_THRESHOLD or (
-            count % settings.THUMBNAIL_ERROR_REPEATED_ALERT_FREQUENCY == 0
-        ):
-            with push_scope() as scope:
-                set_context(
-                    "upstream_url",
-                    {
-                        "url": upstream_url,
-                        "params": params,
-                        "headers": headers,
-                    },
-                )
-                scope.set_tag(
-                    "occurrences", settings.THUMBNAIL_ERROR_REPEATED_ALERT_FREQUENCY
-                )
-                sentry_sdk.capture_exception(exc)
+
         if isinstance(exc, ClientResponseError):
             status = exc.status
             do_not_wait_for(

api/conf/settings/thumbnails.py

@@ -15,11 +15,3 @@
 # to cast them in assertions to match the parsed param types)
 THUMBNAIL_WIDTH_PX = config("THUMBNAIL_WIDTH_PX", default="600")
 THUMBNAIL_QUALITY = config("THUMBNAIL_JPG_QUALITY", default="80")
-
-THUMBNAIL_ERROR_INITIAL_ALERT_THRESHOLD = config(
-    "THUMBNAIL_ERROR_INITIAL_ALERT_THRESHOLD", default=100, cast=int
-)
-
-THUMBNAIL_ERROR_REPEATED_ALERT_FREQUENCY = config(
-    "THUMBNAIL_ERROR_REPEATED_ALERT_FREQUENCY", default=1000, cast=int
-)

api/test/unit/utils/test_image_proxy.py

@@ -331,18 +331,8 @@ def test_get_successful_records_response_code(
 
 
 alert_count_params = pytest.mark.parametrize(
-    "count_start, should_alert",
-    [
-        (0, True),
-        (1, True),
-        (50, True),
-        (99, True),
-        (100, False),
-        (999, True),
-        (1000, False),
-        (1500, False),
-        (1999, True),
-    ],
+    "count_start",
+    [0, 1, 50, 99, 100, 999, 1000, 1500, 1999],
 )
 
 MOCK_CONNECTION_KEY = ConnectionKey(
@@ -385,7 +375,6 @@ def test_get_exception_handles_error(
     exc,
     exc_name,
     count_start,
-    should_alert,
     sentry_capture_exception,
     setup_request_exception,
     is_cache_reachable,
@@ -409,12 +398,7 @@ def test_get_exception_handles_error(
     ):
         photon_get(TEST_MEDIA_INFO)
 
-    assert_func = (
-        sentry_capture_exception.assert_called_once
-        if should_alert
-        else sentry_capture_exception.assert_not_called
-    )
-    assert_func()
+    sentry_capture_exception.assert_not_called()
 
     if is_cache_reachable:
         assert cache.get(key) == str(count_start + 1).encode()
@@ -438,7 +422,6 @@ def test_get_http_exception_handles_error(
     status_code,
     text,
     count_start,
-    should_alert,
     sentry_capture_exception,
     is_cache_reachable,
     cache_name,
@@ -459,12 +442,7 @@ def test_get_http_exception_handles_error(
             pook.get(PHOTON_URL_FOR_TEST_IMAGE).reply(status_code, text)
             photon_get(TEST_MEDIA_INFO)
 
-    assert_func = (
-        sentry_capture_exception.assert_called_once
-        if should_alert
-        else sentry_capture_exception.assert_not_called
-    )
-    assert_func()
+    sentry_capture_exception.assert_not_called()
 
     if is_cache_reachable:
         assert cache.get(key) == str(count_start + 1).encode()