Commit 93698625 authored by Jan Reimes's avatar Jan Reimes
Browse files

♻️ refactor(config): add ValidationInfo type hint to adobe_requires_both_credentials

parent 8b11d596
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -4,7 +4,7 @@ from __future__ import annotations

from pathlib import Path

from pydantic import Field, field_validator
from pydantic import Field, ValidationInfo, field_validator
from pydantic_settings import BaseSettings, SettingsConfigDict


@@ -43,7 +43,7 @@ class ProviderSettings(BaseSettings):

    @field_validator("adobe_client_secret")
    @classmethod
    def adobe_requires_both_credentials(cls, v: str | None, info) -> str | None:
    def adobe_requires_both_credentials(cls, v: str | None, info: ValidationInfo) -> str | None:
        """Validate Adobe requires both client_id and client_secret."""
        client_id = info.data.get("adobe_client_id")
        if client_id and not v:
+1 −2
Original line number Diff line number Diff line
@@ -15,6 +15,7 @@ from pdf_remote_converter.exceptions import (
    QuotaExceededError,
    RateLimitError,
)
from pdf_remote_converter.http import create_uncached_client
from pdf_remote_converter.providers import get_provider
from pdf_remote_converter.providers.base import ProviderBackend
from pdf_remote_converter.providers.models import ConversionResult
@@ -92,8 +93,6 @@ def convert_with_failover(
    Raises:
        AllProvidersFailedError: When all providers fail conversion.
    """
    from pdf_remote_converter.http import create_uncached_client

    # If force flag is set, use uncached HTTP client for all providers
    if force:
        for provider in providers: