Removing Sentry

Author: aweiland

.github/workflows/_deploy.yaml

@@ -86,7 +86,6 @@ jobs:
           environment-variables: |
             VERSION=${{ inputs.image-tag }}
             DD_VERSION=${{ inputs.image-tag }}
-            SENTRY_RELEASE=${{ inputs.image-tag }}
             COMMIT_ID=${{ github.sha }}
           docker-labels: |
             com.datadoghq.tags.version=${{ inputs.image-tag }}

.github/workflows/_migrations.yaml

@@ -58,7 +58,6 @@ jobs:
           environment-variables: |
             VERSION=${{ inputs.image-tag }}
             DD_VERSION=${{ inputs.image-tag }}
-            SENTRY_RELEASE=${{ inputs.image-tag }}
           docker-labels: |
             com.datadoghq.tags.version=${{ inputs.image-tag }}
 

pyproject.toml

@@ -31,7 +31,6 @@ dependencies = [
   "python-multipart>=0.0.31",
   "python-slugify==8.0.4",
   "redis==5.2.*",
-  "sentry-sdk~=2.13",
   "sqlalchemy[asyncio]~=1.4.53",
   "sqlalchemy-utils~=0.41.2",
   "structlog~=25.4.0",

src/apps/answers/service.py

@@ -12,7 +12,6 @@
 from typing import Callable, List, Mapping
 
 import aiohttp
-import sentry_sdk
 from cryptography.hazmat.backends import default_backend
 from cryptography.hazmat.primitives import hashes
 from cryptography.hazmat.primitives.asymmetric import padding
@@ -1727,7 +1726,7 @@ async def _create_alerts(
                     ).model_dump(),
                 )
             except Exception as e:
-                sentry_sdk.capture_exception(e)
+                logger.exception(str(e))
                 break
         await self.send_alert_mail(persons)
 

src/apps/answers/tasks.py

@@ -1,9 +1,7 @@
 import base64
 import io
-import traceback
 import uuid
 
-import sentry_sdk
 from fastapi import UploadFile
 from sqlalchemy.ext.asyncio import AsyncSession
 
@@ -13,6 +11,7 @@
 from apps.mailing.services import MailingService
 from broker import broker
 from infrastructure.database import session_manager
+from infrastructure.logger import logger
 
 # moved from previous implementation
 
@@ -64,5 +63,4 @@ async def create_report(
                 )
             )
     except Exception as e:
-        traceback.print_exception(e)
-        sentry_sdk.capture_exception(e)
+        logger.exception(str(e))

src/apps/integrations/loris/service/loris.py

@@ -7,7 +7,6 @@
 from collections import defaultdict
 
 import aiohttp
-import sentry_sdk
 from pydantic.json import pydantic_encoder
 
 from apps.activities.crud.activity_history import ActivityHistoriesCRUD
@@ -890,7 +889,7 @@ async def _create_integration_alerts(self, applet_id: uuid.UUID, message: str):
                 )
 
             except Exception as e:
-                sentry_sdk.capture_exception(e)
+                logger.exception(str(e))
                 break
 
     async def create_loris_integration(self, hostname, username, project, password) -> LorisIntegration:

src/config/__init__.py

@@ -23,7 +23,6 @@
 from config.rabbitmq import RabbitMQSettings
 from config.redis import RedisSettings
 from config.secret import SecretSettings
-from config.sentry import SentrySettings
 from config.service import JsonLdConverterSettings, ServiceSettings
 from config.superuser import SuperAdmin
 from config.task import AnswerEncryption, AudioFileConvert, ImageConvert
@@ -71,9 +70,6 @@ class Settings(BaseSettings):
     # CDN configs
     cdn: CDNSettings = CDNSettings()
 
-    # Sentry stuff
-    sentry: SentrySettings = SentrySettings()
-
     # FCM Notification configs
     fcm: FirebaseCloudMessagingSettings = FirebaseCloudMessagingSettings()
 

src/config/sentry.py

@@ -1,6 +1,5 @@
 from typing import Iterable, Type
 
-import sentry_sdk
 from asgi_correlation_id import CorrelationIdMiddleware
 from asyncpg import InvalidPasswordError
 from fastapi import FastAPI
@@ -111,9 +110,6 @@ def create_app():
         lifespan=lifespan,
     )
 
-    if settings.sentry.dsn:
-        sentry_sdk.init(dsn=settings.sentry.dsn, traces_sample_rate=1.0)
-
     # Include routers
     for router in routers:
         app.include_router(router)

src/infrastructure/app.py

@@ -5,9 +5,9 @@
 
 import redis.asyncio as redis
 from redis.typing import EncodableT
-from sentry_sdk import capture_exception
 
 from config import settings
+from infrastructure.logger import logger
 
 
 class RedisCacheTest:
@@ -139,11 +139,8 @@ def _start(self):
                 db=self.db,
                 **self.configuration,
             )
-        except redis.exceptions.ConnectionError as e:
-            try:
-                capture_exception(e)
-            except ImportError:
-                print(e)
+        except redis.ConnectionError as e:
+            logger.exception(str(e))
 
     async def get(self, key: str) -> typing.Optional[str]:
         if not self._cache: