Handle missing Docker config file mounts

Dockerfile

@@ -31,6 +31,7 @@ COPY pyproject.toml uv.lock ./
 RUN uv sync --frozen --no-dev --no-install-project
 
 COPY main.py ./
+COPY config.example.json ./
 COPY VERSION ./
 COPY services ./services
 COPY --from=web-build /app/web/out ./web_dist

README.md

@@ -74,6 +74,8 @@ git clone git@github.com:basketikun/chatgpt2api.git
 docker compose up -d
 ```
 
+如果之前在宿主机上没有 `config.json` 就直接执行了 `docker compose up -d`，Docker 可能会错误地创建一个 `config.json/` 目录。遇到这种情况请先删除这个目录，再重新创建 `config.json` 文件后启动。
+
 ## 社区支持
 学 AI , 上 L 站
 

services/config.py

@@ -3,6 +3,7 @@ from __future__ import annotations
 import json
 import os
 import shutil
+import sys
 from dataclasses import dataclass
 from pathlib import Path
 from typing import cast
@@ -22,6 +23,19 @@ class AppSettings:
     accounts_file: Path
     refresh_account_interval_minute: int
 
+
+def _readable_json_file(path: Path, *, name: str) -> Path | None:
+    if not path.exists():
+        return None
+    if path.is_dir():
+        print(
+            f"Warning: {name} at '{path}' is a directory, ignoring it and falling back to other configuration sources.",
+            file=sys.stderr,
+        )
+        return None
+    return path
+
+
 def _load_json_object(path: Path, *, name: str) -> dict[str, object]:
     text = path.read_text(encoding="utf-8").strip()
     if not text:
@@ -36,9 +50,10 @@ def _load_json_object(path: Path, *, name: str) -> dict[str, object]:
 def _ensure_config_file() -> None:
     if CONFIG_FILE.exists():
         return
-    if not CONFIG_EXAMPLE_FILE.exists():
+    example_file = _readable_json_file(CONFIG_EXAMPLE_FILE, name="config.example.json")
+    if example_file is None:
         return
-    shutil.copyfile(CONFIG_EXAMPLE_FILE, CONFIG_FILE)
+    shutil.copyfile(example_file, CONFIG_FILE)
 
 
 def _load_settings() -> AppSettings:
@@ -46,10 +61,13 @@ def _load_settings() -> AppSettings:
     _ensure_config_file()
 
     raw_config: dict[str, object] = {}
-    if CONFIG_EXAMPLE_FILE.exists():
-        raw_config.update(_load_json_object(CONFIG_EXAMPLE_FILE, name="config.example.json"))
-    if CONFIG_FILE.exists():
-        raw_config.update(_load_json_object(CONFIG_FILE, name="config.json"))
+    example_file = _readable_json_file(CONFIG_EXAMPLE_FILE, name="config.example.json")
+    if example_file is not None:
+        raw_config.update(_load_json_object(example_file, name="config.example.json"))
+
+    config_file = _readable_json_file(CONFIG_FILE, name="config.json")
+    if config_file is not None:
+        raw_config.update(_load_json_object(config_file, name="config.json"))
 
     auth_key = str(os.getenv("CHATGPT2API_AUTH_KEY") or raw_config.get("auth-key") or "").strip()
     if not auth_key:

test/test_config.py

@@ -0,0 +1,64 @@
+import json
+import tempfile
+import unittest
+from pathlib import Path
+
+
+ROOT_DIR = Path(__file__).resolve().parents[1]
+ROOT_CONFIG_FILE = ROOT_DIR / "config.json"
+
+
+class ConfigLoadingTests(unittest.TestCase):
+    @classmethod
+    def setUpClass(cls) -> None:
+        cls._created_root_config = False
+        if not ROOT_CONFIG_FILE.exists():
+            ROOT_CONFIG_FILE.write_text(json.dumps({"auth-key": "test-auth"}), encoding="utf-8")
+            cls._created_root_config = True
+
+        from services import config as config_module
+
+        cls.config_module = config_module
+
+    @classmethod
+    def tearDownClass(cls) -> None:
+        if cls._created_root_config and ROOT_CONFIG_FILE.exists():
+            ROOT_CONFIG_FILE.unlink()
+
+    def test_load_settings_falls_back_to_example_when_config_path_is_directory(self) -> None:
+        with tempfile.TemporaryDirectory() as tmp_dir:
+            base_dir = Path(tmp_dir)
+            data_dir = base_dir / "data"
+            config_dir = base_dir / "config.json"
+            example_file = base_dir / "config.example.json"
+
+            config_dir.mkdir()
+            example_file.write_text(
+                json.dumps({"auth-key": "example-auth", "refresh_account_interval_minute": 15}),
+                encoding="utf-8",
+            )
+
+            module = self.config_module
+            old_base_dir = module.BASE_DIR
+            old_data_dir = module.DATA_DIR
+            old_config_file = module.CONFIG_FILE
+            old_config_example_file = module.CONFIG_EXAMPLE_FILE
+            try:
+                module.BASE_DIR = base_dir
+                module.DATA_DIR = data_dir
+                module.CONFIG_FILE = config_dir
+                module.CONFIG_EXAMPLE_FILE = example_file
+
+                settings = module._load_settings()
+
+                self.assertEqual(settings.auth_key, "example-auth")
+                self.assertEqual(settings.refresh_account_interval_minute, 15)
+            finally:
+                module.BASE_DIR = old_base_dir
+                module.DATA_DIR = old_data_dir
+                module.CONFIG_FILE = old_config_file
+                module.CONFIG_EXAMPLE_FILE = old_config_example_file
+
+
+if __name__ == "__main__":
+    unittest.main()