From c1ff0a00dbf5ae9dabf94008f143a8bc20d6f63c Mon Sep 17 00:00:00 2001 From: Samir Boulahtit Date: Sat, 22 Nov 2025 15:59:15 +0100 Subject: [PATCH] fix: correct authentication dependency names in CMS API endpoints MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fix ImportError by using correct authentication dependency names: - Use get_current_admin_api instead of get_current_admin_user - Use get_current_vendor_api instead of get_current_vendor_user These are the correct dependency names for API endpoints that require Authorization header authentication (no cookie support). Fixes: ImportError: cannot import name 'get_current_admin_user' 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- app/api/v1/admin/content_pages.py | 14 +++++++------- app/api/v1/vendor/content_pages.py | 14 +++++++------- 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/app/api/v1/admin/content_pages.py b/app/api/v1/admin/content_pages.py index 1d687bed..2881973e 100644 --- a/app/api/v1/admin/content_pages.py +++ b/app/api/v1/admin/content_pages.py @@ -14,7 +14,7 @@ from fastapi import APIRouter, Depends, HTTPException, Query from pydantic import BaseModel, Field from sqlalchemy.orm import Session -from app.api.deps import get_current_admin_user, get_db +from app.api.deps import get_current_admin_api, get_db from app.services.content_page_service import content_page_service from models.database.user import User @@ -85,7 +85,7 @@ class ContentPageResponse(BaseModel): @router.get("/platform", response_model=List[ContentPageResponse]) def list_platform_pages( include_unpublished: bool = Query(False, description="Include draft pages"), - current_user: User = Depends(get_current_admin_user), + current_user: User = Depends(get_current_admin_api), db: Session = Depends(get_db) ): """ @@ -104,7 +104,7 @@ def list_platform_pages( @router.post("/platform", response_model=ContentPageResponse, status_code=201) def create_platform_page( page_data: ContentPageCreate, - current_user: User = Depends(get_current_admin_user), + current_user: User = Depends(get_current_admin_api), db: Session = Depends(get_db) ): """ @@ -140,7 +140,7 @@ def create_platform_page( def list_all_pages( vendor_id: Optional[int] = Query(None, description="Filter by vendor ID"), include_unpublished: bool = Query(False, description="Include draft pages"), - current_user: User = Depends(get_current_admin_user), + current_user: User = Depends(get_current_admin_api), db: Session = Depends(get_db) ): """ @@ -176,7 +176,7 @@ def list_all_pages( @router.get("/{page_id}", response_model=ContentPageResponse) def get_page( page_id: int, - current_user: User = Depends(get_current_admin_user), + current_user: User = Depends(get_current_admin_api), db: Session = Depends(get_db) ): """Get a specific content page by ID.""" @@ -192,7 +192,7 @@ def get_page( def update_page( page_id: int, page_data: ContentPageUpdate, - current_user: User = Depends(get_current_admin_user), + current_user: User = Depends(get_current_admin_api), db: Session = Depends(get_db) ): """Update a content page (platform or vendor).""" @@ -220,7 +220,7 @@ def update_page( @router.delete("/{page_id}", status_code=204) def delete_page( page_id: int, - current_user: User = Depends(get_current_admin_user), + current_user: User = Depends(get_current_admin_api), db: Session = Depends(get_db) ): """Delete a content page.""" diff --git a/app/api/v1/vendor/content_pages.py b/app/api/v1/vendor/content_pages.py index 68f8a418..b9af5302 100644 --- a/app/api/v1/vendor/content_pages.py +++ b/app/api/v1/vendor/content_pages.py @@ -14,7 +14,7 @@ from fastapi import APIRouter, Depends, HTTPException, Query from pydantic import BaseModel, Field from sqlalchemy.orm import Session -from app.api.deps import get_current_vendor_user, get_db +from app.api.deps import get_current_vendor_api, get_db from app.services.content_page_service import content_page_service from models.database.user import User @@ -84,7 +84,7 @@ class ContentPageResponse(BaseModel): @router.get("/", response_model=List[ContentPageResponse]) def list_vendor_pages( include_unpublished: bool = Query(False, description="Include draft pages"), - current_user: User = Depends(get_current_vendor_user), + current_user: User = Depends(get_current_vendor_api), db: Session = Depends(get_db) ): """ @@ -107,7 +107,7 @@ def list_vendor_pages( @router.get("/overrides", response_model=List[ContentPageResponse]) def list_vendor_overrides( include_unpublished: bool = Query(False, description="Include draft pages"), - current_user: User = Depends(get_current_vendor_user), + current_user: User = Depends(get_current_vendor_api), db: Session = Depends(get_db) ): """ @@ -131,7 +131,7 @@ def list_vendor_overrides( def get_page( slug: str, include_unpublished: bool = Query(False, description="Include draft pages"), - current_user: User = Depends(get_current_vendor_user), + current_user: User = Depends(get_current_vendor_api), db: Session = Depends(get_db) ): """ @@ -158,7 +158,7 @@ def get_page( @router.post("/", response_model=ContentPageResponse, status_code=201) def create_vendor_page( page_data: VendorContentPageCreate, - current_user: User = Depends(get_current_vendor_user), + current_user: User = Depends(get_current_vendor_api), db: Session = Depends(get_db) ): """ @@ -192,7 +192,7 @@ def create_vendor_page( def update_vendor_page( page_id: int, page_data: VendorContentPageUpdate, - current_user: User = Depends(get_current_vendor_user), + current_user: User = Depends(get_current_vendor_api), db: Session = Depends(get_db) ): """ @@ -233,7 +233,7 @@ def update_vendor_page( @router.delete("/{page_id}", status_code=204) def delete_vendor_page( page_id: int, - current_user: User = Depends(get_current_vendor_user), + current_user: User = Depends(get_current_vendor_api), db: Session = Depends(get_db) ): """