refactored sentry app

Author: nvo87

src/__init__.py

@@ -1,14 +1,18 @@
+import os
+from distutils.util import strtobool
 from functools import lru_cache
 
 from pydantic import BaseSettings
 
 
 class Settings(BaseSettings):
+    DEBUG: bool = bool(strtobool(os.getenv('DEBUG', 'False')))
+
     BASECAMP_ACCOUNT_ID: str = 'bc-acc-id'
     BASECAMP_CHATBOT_KEY: str = 'bc-bot-key'
 
     class Config:
-        env_file = '.env'
+        env_file = 'src/.env'
 
 
 @lru_cache()

src/config.py

@@ -0,0 +1,13 @@
+from fastapi import FastAPI
+
+from config import get_settings
+from sentry.api import api_router
+
+
+settings = get_settings()
+
+app = FastAPI(
+    debug=settings.DEBUG,
+)
+
+app.include_router(api_router)

src/main.py

@@ -1,27 +1,16 @@
-from typing import Any, Mapping
 from fastapi.requests import Request
 from fastapi.responses import Response
-from uuid import UUID
 
-from fastapi import FastAPI
 from fastapi import status
-from pydantic.config import Extra
-from pydantic.main import BaseModel
+from fastapi import APIRouter
 
-app = FastAPI()
+from sentry.types import Webhook
 
 
-class Installation(BaseModel):
-    uuid: UUID
+api_router = APIRouter()
 
 
-class Webhook(BaseModel, extra=Extra.allow):
-    action: str
-    data: Mapping[str, Any]
-    installation: Installation
-
-
-@app.post("/api/sentry/webhook", status_code=status.HTTP_200_OK)
+@api_router.post("/api/sentry/webhook", status_code=status.HTTP_200_OK)
 async def sentry_webhook(webhook: Webhook, request: Request):
     if request.headers.get('sentry-hook-resource') == 'issue':
 

src/sentry/__init__.py

@@ -1,6 +1,6 @@
 import pytest
 from fastapi.testclient import TestClient
-from sentry.main import app
+from main import app
 from sentry.tests.request_data import event, issue
 
 

src/sentry/main.py renamed to src/sentry/api.py

@@ -0,0 +1,15 @@
+from typing import Any, Mapping
+from uuid import UUID
+
+from pydantic import BaseModel
+from pydantic.config import Extra
+
+
+class Installation(BaseModel):
+    uuid: UUID
+
+
+class Webhook(BaseModel, extra=Extra.allow):
+    action: str
+    data: Mapping[str, Any]
+    installation: Installation