Daniel Bedeleanu
ea49cd6e4a
- Create backend/services/image_storage.py with 4 core functions:
- sanitize_filename(): remove unsafe chars, limit to 255 chars, convert to lowercase
- get_unique_filename(): handle collisions with UUID suffix (format: {name}_{uuid8}_{variant}.jpg)
- ensure_image_directories(): create /images/ root and category subdirs on startup
- save_image(): save bytes to /images/{category}/{filename}, returns relative path
- Create comprehensive test suite (22 tests) covering all functionality
- Integrate ensure_image_directories() into FastAPI startup event
- Directory structure: /images/{category}/{filename}
- Collision handling: auto-suffix with UUID if filename exists
- All tests passing, pathlib.Path for safe operations
74 lines
2.7 KiB
Python
74 lines
2.7 KiB
Python
from __future__ import annotations
|
|
|
|
from collections.abc import Mapping
|
|
from typing import Any
|
|
|
|
from starlette._exception_handler import (
|
|
ExceptionHandlers,
|
|
StatusHandlers,
|
|
wrap_app_handling_exceptions,
|
|
)
|
|
from starlette.exceptions import HTTPException, WebSocketException
|
|
from starlette.requests import Request
|
|
from starlette.responses import PlainTextResponse, Response
|
|
from starlette.types import ASGIApp, ExceptionHandler, Receive, Scope, Send
|
|
from starlette.websockets import WebSocket
|
|
|
|
|
|
class ExceptionMiddleware:
|
|
def __init__(
|
|
self,
|
|
app: ASGIApp,
|
|
handlers: Mapping[Any, ExceptionHandler] | None = None,
|
|
debug: bool = False,
|
|
) -> None:
|
|
self.app = app
|
|
self.debug = debug # TODO: We ought to handle 404 cases if debug is set.
|
|
self._status_handlers: StatusHandlers = {}
|
|
self._exception_handlers: ExceptionHandlers = {
|
|
HTTPException: self.http_exception,
|
|
WebSocketException: self.websocket_exception,
|
|
}
|
|
if handlers is not None: # pragma: no branch
|
|
for key, value in handlers.items():
|
|
self.add_exception_handler(key, value)
|
|
|
|
def add_exception_handler(
|
|
self,
|
|
exc_class_or_status_code: int | type[Exception],
|
|
handler: ExceptionHandler,
|
|
) -> None:
|
|
if isinstance(exc_class_or_status_code, int):
|
|
self._status_handlers[exc_class_or_status_code] = handler
|
|
else:
|
|
assert issubclass(exc_class_or_status_code, Exception)
|
|
self._exception_handlers[exc_class_or_status_code] = handler
|
|
|
|
async def __call__(self, scope: Scope, receive: Receive, send: Send) -> None:
|
|
if scope["type"] not in ("http", "websocket"):
|
|
await self.app(scope, receive, send)
|
|
return
|
|
|
|
scope["starlette.exception_handlers"] = (
|
|
self._exception_handlers,
|
|
self._status_handlers,
|
|
)
|
|
|
|
conn: Request | WebSocket
|
|
if scope["type"] == "http":
|
|
conn = Request(scope, receive, send)
|
|
else:
|
|
conn = WebSocket(scope, receive, send)
|
|
|
|
await wrap_app_handling_exceptions(self.app, conn)(scope, receive, send)
|
|
|
|
async def http_exception(self, request: Request, exc: Exception) -> Response:
|
|
assert isinstance(exc, HTTPException)
|
|
if exc.status_code in {204, 304}:
|
|
return Response(status_code=exc.status_code, headers=exc.headers)
|
|
return PlainTextResponse(exc.detail, status_code=exc.status_code, headers=exc.headers)
|
|
|
|
async def websocket_exception(self, websocket: WebSocket, exc: Exception) -> None:
|
|
assert isinstance(exc, WebSocketException)
|
|
await websocket.close(code=exc.code, reason=exc.reason) # pragma: no cover
|