wangbo/ComfyUI
1
0
Fork 0
mirror of https://github.com/comfyanonymous/ComfyUI.git synced 2026-05-05 14:52:46 +08:00
Code Issues Actions 22 Packages Projects Releases Wiki Activity

Require at least one tag in UploadAssetSpec

Browse Source 
Enforce non-empty tags at the Pydantic validation layer so uploads
with no tags are rejected with a 400 before reaching ingest. Adds
test_upload_empty_tags_rejected to cover this case.

Amp-Thread-ID: https://ampcode.com/threads/T-019ce377-8bde-7048-bc28-a9df063409f9
Co-authored-by: Amp <amp@ampcode.com>
This commit is contained in:
Luke Mino-Altherr 2026-03-12 12:59:22 -07:00
parent aa953a2fdc
commit 9cfb9c7a9d
2 changed files with 19 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
app/assets/api/schemas_in.py
View File

@ -333,7 +333,8 @@ class UploadAssetSpec(BaseModel):
@model_validator(mode="after") @model_validator(mode="after")
def _validate_order(self): def _validate_order(self):
if self.tags: if not self.tags:
raise ValueError("at least one tag is required for uploads")
root = self.tags[0] root = self.tags[0]
if root not in {"models", "input", "output"}: if root not in {"models", "input", "output"}:
raise ValueError("first tag must be one of: models, input, output") raise ValueError("first tag must be one of: models, input, output")

9
tests-unit/assets_test/test_uploads.py
View File

@ -243,6 +243,15 @@ def test_upload_tags_traversal_guard(http: requests.Session, api_base: str):
assert body["error"]["code"] in ("BAD_REQUEST", "INVALID_BODY") assert body["error"]["code"] in ("BAD_REQUEST", "INVALID_BODY")
def test_upload_empty_tags_rejected(http: requests.Session, api_base: str):
files = {"file": ("notags.bin", b"A" * 64, "application/octet-stream")}
form = {"tags": json.dumps([]), "name": "notags.bin", "user_metadata": json.dumps({})}
r = http.post(api_base + "/api/assets", data=form, files=files, timeout=120)
body = r.json()
assert r.status_code == 400
assert body["error"]["code"] == "INVALID_BODY"
@pytest.mark.parametrize("root", ["input", "output"]) @pytest.mark.parametrize("root", ["input", "output"])
def test_duplicate_upload_same_display_name_does_not_clobber( def test_duplicate_upload_same_display_name_does_not_clobber(
root: str, root: str,