chore: add configurable expiry for management API tokens

app/api/auth.py

@@ -4,7 +4,7 @@
 import os
 import time
 import uuid
-from datetime import datetime
+from datetime import datetime, timedelta, UTC
 import email_validator
 
 from typing import Optional, List, Union
@@ -36,7 +36,7 @@
 from app.core.worker import generate_pricing_url
 
 from app.db.database import get_db
-from app.db.models import DBUser, DBAPIToken, DBRegion, DBTeam
+from app.db.models import DBUser, DBAPIToken, DBRegion, DBTeam, DBAPITokenExpiryOption
 
 from app.services.litellm import LiteLLMService
 from app.services.dynamodb import DynamoDBService
@@ -50,6 +50,7 @@
     APIToken,
     APITokenCreate,
     APITokenResponse,
+    APITokenExpiryOption,
     UserUpdate,
     EmailValidation,
     LoginData,
@@ -519,6 +520,20 @@ def generate_api_token() -> str:
     return secrets.token_urlsafe(32)
 
 
+@router.get("/token/expiry-options", response_model=List[APITokenExpiryOption])
+async def list_expiry_options(
+    current_user=Depends(get_current_user_from_auth),
+    db: Session = Depends(get_db),
+):
+    """List available API token expiry options"""
+    return (
+        db.query(DBAPITokenExpiryOption)
+        .filter(DBAPITokenExpiryOption.is_active)
+        .order_by(DBAPITokenExpiryOption.id)
+        .all()
+    )
+
+
 @router.post("/token", response_model=APIToken)
 async def create_token(
     token_create: APITokenCreate,
@@ -552,8 +567,34 @@ async def create_token(
         # Create token for the current user
         user_id = current_user.id
 
+    # Fetch expiry option from DB (only active options are valid)
+    expiry_slug = token_create.expiry or "forever"
+    db_expiry_opt = (
+        db.query(DBAPITokenExpiryOption)
+        .filter(
+            DBAPITokenExpiryOption.slug == expiry_slug,
+            DBAPITokenExpiryOption.is_active,
+        )
+        .first()
+    )
+
+    if not db_expiry_opt:
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail=f"Invalid or inactive expiry option: {expiry_slug}",
+        )
+
+    # Calculate expiration date
+    expires_at = None
+    if db_expiry_opt.days is not None:
+        expires_at = datetime.now(UTC) + timedelta(days=db_expiry_opt.days)
+
     db_token = DBAPIToken(
-        name=token_create.name, token=generate_api_token(), user_id=user_id
+        name=token_create.name,
+        token=generate_api_token(),
+        user_id=user_id,
+        expires_at=expires_at,
+        expiry_option=db_expiry_opt.slug,
     )
     db.add(db_token)
     db.commit()

app/core/security.py

@@ -121,12 +121,24 @@ async def get_current_user_from_auth(
     try:
         db_token = db.query(DBAPIToken).filter(DBAPIToken.token == token_to_try).first()
         if db_token:
+            # Check if token is expired
+            if db_token.expires_at and db_token.expires_at < datetime.now(UTC):
+                raise HTTPException(
+                    status_code=status.HTTP_401_UNAUTHORIZED,
+                    detail="API token has expired",
+                    headers={"WWW-Authenticate": "Bearer"},
+                )
+
             # Update last used timestamp
             db_token.last_used_at = datetime.now(UTC)
             db.commit()
             return db_token.owner
+    except HTTPException:
+        raise
     except Exception:
-        pass
+        logger.exception(
+            "Unexpected error during API token validation; falling back to JWT validation"
+        )
 
     # If API token validation fails, try JWT validation
     try:

app/db/init_db.py

@@ -1,11 +1,63 @@
-from app.db.models import Base
-from app.db.database import engine
+import logging
+
+from app.db.models import Base, DBAPITokenExpiryOption
+from app.db.database import engine, SessionLocal
+
+logger = logging.getLogger(__name__)
+
+
+def init_api_token_expiry_options(db=None):
+    logger.info("Initializing API token expiry options...")
+    own_db = False
+    if db is None:
+        db = SessionLocal()
+        own_db = True
+
+    try:
+        options = [
+            {"name": "1 day", "slug": "1_day", "days": 1},
+            {"name": "1 week", "slug": "1_week", "days": 7},
+            {"name": "1 month", "slug": "1_month", "days": 30},
+            {"name": "2 months", "slug": "2_months", "days": 60},
+            {"name": "3 months", "slug": "3_months", "days": 90},
+            {"name": "4 months", "slug": "4_months", "days": 120},
+            {"name": "5 months", "slug": "5_months", "days": 150},
+            {"name": "6 months", "slug": "6_months", "days": 180},
+            {"name": "7 months", "slug": "7_months", "days": 210},
+            {"name": "8 months", "slug": "8_months", "days": 240},
+            {"name": "9 months", "slug": "9_months", "days": 270},
+            {"name": "10 months", "slug": "10_months", "days": 300},
+            {"name": "11 months", "slug": "11_months", "days": 330},
+            {"name": "1 year", "slug": "1_year", "days": 365},
+            {"name": "forever", "slug": "forever", "days": None},
+        ]
+
+        for opt_data in options:
+            existing = (
+                db.query(DBAPITokenExpiryOption)
+                .filter(DBAPITokenExpiryOption.slug == opt_data["slug"])
+                .first()
+            )
+            if not existing:
+                db_opt = DBAPITokenExpiryOption(**opt_data)
+                db.add(db_opt)
+
+        db.commit()
+        logger.info("API token expiry options initialized successfully!")
+    except Exception:
+        logger.exception("Error initializing API token expiry options")
+        db.rollback()
+        raise
+    finally:
+        if own_db:
+            db.close()
 
 
 def init_db():
-    print("Creating database tables...")
+    logger.info("Creating database tables...")
     Base.metadata.create_all(bind=engine)
-    print("Database tables created successfully!")
+    logger.info("Database tables created successfully!")
+    init_api_token_expiry_options()
 
 
 if __name__ == "__main__":

app/db/models.py

@@ -98,6 +98,16 @@ class DBRegion(Base):
     teams = relationship("DBTeamRegion", back_populates="region")
 
 
+class DBAPITokenExpiryOption(Base):
+    __tablename__ = "api_token_expiry_options"
+
+    id = Column(Integer, primary_key=True, index=True)
+    name = Column(String, nullable=False)
+    slug = Column(String, unique=True, index=True, nullable=False)
+    days = Column(Integer, nullable=True)  # None for forever
+    is_active = Column(Boolean, default=True)
+
+
 class DBAPIToken(Base):
     __tablename__ = "api_tokens"
 
@@ -106,6 +116,8 @@ class DBAPIToken(Base):
     token = Column(String, unique=True, index=True)
     created_at = Column(DateTime(timezone=True), default=func.now())
     last_used_at = Column(DateTime(timezone=True), nullable=True)
+    expires_at = Column(DateTime(timezone=True), nullable=True)
+    expiry_option = Column(String, default="forever", nullable=False)
     user_id = Column(Integer, ForeignKey("users.id"))
 
     owner = relationship("DBUser", back_populates="api_tokens")

app/migrations/versions/20260408_125946_daf5bf0b03c2_add_api_token_expiry_options.py

@@ -0,0 +1,110 @@
+"""add_api_token_expiry_options
+
+Revision ID: daf5bf0b03c2
+Revises: a1b2c3d4e5f6
+Create Date: 2026-04-08 12:59:46.600703+00:00
+
+"""
+
+from typing import Sequence, Union
+
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision: str = "daf5bf0b03c2"
+down_revision: Union[str, None] = "a1b2c3d4e5f6"
+branch_labels: Union[str, Sequence[str], None] = None
+depends_on: Union[str, Sequence[str], None] = None
+
+
+def upgrade() -> None:
+    # Create api_token_expiry_options table
+    op.create_table(
+        "api_token_expiry_options",
+        sa.Column("id", sa.Integer(), nullable=False),
+        sa.Column("name", sa.String(), nullable=False),
+        sa.Column("slug", sa.String(), nullable=False),
+        sa.Column("days", sa.Integer(), nullable=True),
+        sa.Column("is_active", sa.Boolean(), nullable=False, server_default=sa.true()),
+        sa.PrimaryKeyConstraint("id"),
+    )
+    op.create_index(
+        op.f("ix_api_token_expiry_options_id"),
+        "api_token_expiry_options",
+        ["id"],
+        unique=False,
+    )
+    op.create_index(
+        op.f("ix_api_token_expiry_options_slug"),
+        "api_token_expiry_options",
+        ["slug"],
+        unique=True,
+    )
+
+    # Seed default expiry options so the API is usable immediately after migration
+    expiry_options_table = sa.table(
+        "api_token_expiry_options",
+        sa.column("name", sa.String()),
+        sa.column("slug", sa.String()),
+        sa.column("days", sa.Integer()),
+        sa.column("is_active", sa.Boolean()),
+    )
+    op.bulk_insert(
+        expiry_options_table,
+        [
+            {"name": "1 day", "slug": "1_day", "days": 1, "is_active": True},
+            {"name": "1 week", "slug": "1_week", "days": 7, "is_active": True},
+            {"name": "1 month", "slug": "1_month", "days": 30, "is_active": True},
+            {"name": "2 months", "slug": "2_months", "days": 60, "is_active": True},
+            {"name": "3 months", "slug": "3_months", "days": 90, "is_active": True},
+            {"name": "4 months", "slug": "4_months", "days": 120, "is_active": True},
+            {"name": "5 months", "slug": "5_months", "days": 150, "is_active": True},
+            {"name": "6 months", "slug": "6_months", "days": 180, "is_active": True},
+            {"name": "7 months", "slug": "7_months", "days": 210, "is_active": True},
+            {"name": "8 months", "slug": "8_months", "days": 240, "is_active": True},
+            {"name": "9 months", "slug": "9_months", "days": 270, "is_active": True},
+            {"name": "10 months", "slug": "10_months", "days": 300, "is_active": True},
+            {"name": "11 months", "slug": "11_months", "days": 330, "is_active": True},
+            {"name": "1 year", "slug": "1_year", "days": 365, "is_active": True},
+            {"name": "forever", "slug": "forever", "days": None, "is_active": True},
+        ],
+    )
+
+    # Update api_tokens table - these might already exist in some environments but let's ensure they are there
+    # Check if columns exist first to be safe
+    conn = op.get_bind()
+    inspector = sa.inspect(conn)
+    columns = [c["name"] for c in inspector.get_columns("api_tokens")]
+
+    if "expires_at" not in columns:
+        op.add_column(
+            "api_tokens",
+            sa.Column("expires_at", sa.DateTime(timezone=True), nullable=True),
+        )
+    if "expiry_option" not in columns:
+        op.add_column(
+            "api_tokens",
+            sa.Column(
+                "expiry_option", sa.String(), nullable=False, server_default="forever"
+            ),
+        )
+
+
+def downgrade() -> None:
+    conn = op.get_bind()
+    inspector = sa.inspect(conn)
+    columns = [c["name"] for c in inspector.get_columns("api_tokens")]
+
+    if "expiry_option" in columns:
+        op.drop_column("api_tokens", "expiry_option")
+    if "expires_at" in columns:
+        op.drop_column("api_tokens", "expires_at")
+    op.drop_index(
+        op.f("ix_api_token_expiry_options_slug"), table_name="api_token_expiry_options"
+    )
+    op.drop_index(
+        op.f("ix_api_token_expiry_options_id"), table_name="api_token_expiry_options"
+    )
+    op.drop_table("api_token_expiry_options")

app/schemas/models.py

@@ -68,19 +68,31 @@ class User(UserBase):
     audit_logs: ClassVar = relationship("AuditLog", back_populates="user")
 
 
+class APITokenExpiryOption(BaseModel):
+    id: int
+    name: str
+    slug: str
+    days: Optional[int] = None
+    is_active: bool
+    model_config = ConfigDict(from_attributes=True)
+
+
 class APITokenBase(BaseModel):
     name: str
 
 
 class APITokenCreate(APITokenBase):
     user_id: Optional[int] = None
+    expiry: Optional[str] = "forever"
 
 
 class APIToken(APITokenBase):
     id: int
     token: str
     created_at: datetime
     last_used_at: Optional[datetime] = None
+    expires_at: Optional[datetime] = None
+    expiry_option: str
     user_id: int
     model_config = ConfigDict(from_attributes=True)
 
@@ -89,6 +101,8 @@ class APITokenResponse(APITokenBase):
     id: int
     created_at: datetime
     last_used_at: Optional[datetime] = None
+    expires_at: Optional[datetime] = None
+    expiry_option: str
     user_id: int
     model_config = ConfigDict(from_attributes=True)