From fc7e56f71b44e8377f4f807a9b295219cfcda973 Mon Sep 17 00:00:00 2001 From: allen0099 Date: Sat, 26 Sep 2026 17:00:24 +0000 Subject: [PATCH] refactor(session): reuse _stash_session_manager in SessionMiddleware.dispatch SessionMiddleware.dispatch repeated the helper's body inline, so the two copies could drift apart. No behaviour change. Closes #169 --- fastapi_cachex/session/middleware.py | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/fastapi_cachex/session/middleware.py b/fastapi_cachex/session/middleware.py index efc9208..983db0b 100644 --- a/fastapi_cachex/session/middleware.py +++ b/fastapi_cachex/session/middleware.py @@ -220,14 +220,7 @@ async def dispatch( Returns: Response """ - # Store session manager in app state for dependency injection (first request only) - # This allows developers to use get_session_manager() dependency - if not hasattr(request.app.state, "__fastapi_cachex_session_manager"): - setattr( - request.app.state, - "__fastapi_cachex_session_manager", - self.session_manager, - ) + _stash_session_manager(request.app, self.session_manager) # Extract session token from request token = self._extract_token(request)