fix: WebSocket 403 — require_active_plugin skips WebSocket requests

Simpler approach: require_active_plugin._check() now accepts Request
parameter and returns early for WebSocket upgrade requests.
No route splitting needed — all routes stay in their original router.
This commit is contained in:
Agent Zero
2026-07-27 01:34:50 +02:00
parent aae3dc2297
commit 0c67eb0754
2 changed files with 10 additions and 16 deletions
+5 -15
View File
@@ -439,27 +439,17 @@ def create_app() -> FastAPI:
# Skip WebSocket routes — the wrapper breaks WS parameter
# resolution and returns JSONResponse instead of WS close.
from starlette.routing import WebSocketRoute
http_routes = []
ws_routes = []
for route in router.routes:
if isinstance(route, WebSocketRoute):
ws_routes.append(route)
else:
if hasattr(route, 'endpoint'):
route.endpoint = wrap_plugin_route(route.endpoint)
http_routes.append(route)
# Register HTTP routes with active-plugin check
router.routes = http_routes
continue
if hasattr(route, 'endpoint'):
route.endpoint = wrap_plugin_route(route.endpoint)
# Add active-plugin check as a router-level dependency
# (require_active_plugin skips WebSocket requests internally)
app.include_router(
router,
dependencies=[Depends(require_active_plugin(plugin_name))],
)
# Register WebSocket routes WITHOUT active-plugin check
# (WebSocket auth is handled inside the endpoint itself)
if ws_routes:
ws_router = APIRouter()
ws_router.routes = ws_routes
app.include_router(ws_router)
except Exception as exc:
logger.error(f"Failed to register route {route_def.module}.{route_def.router_attr}: {exc}")
break