mirror of
https://github.com/unslothai/unsloth.git
synced 2026-08-13 19:04:10 +00:00
* Studio: return 422 instead of 500 for a binary request body, and bound the traceback a single log record can hold * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Bound the error field, small caps, and validation containers Cap str(exc) under "error" as well as the rendered traceback; clamp the tail so a cap below 2048 cannot make the head negative; bound validation containers, msg and ctx so a large array or a validator that quotes the submitted value cannot round trip unbounded. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Bound keys, error counts, non-finite inputs and the /v1 message path Truncate dictionary keys, iterate with islice instead of materializing every item, cap the number of echoed error records, replace NaN and Infinity (which Starlette refuses to serialize, turning the 422 back into a 500), and run the /v1 envelope's summary through the same sanitizer. * Redact before truncating, and bound the error location filter_sensitive_data now runs before the traceback cap, so a native path cut in half cannot escape redaction; loc elements get the same text budget as input, since a typed mapping copies the offending key into it. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Cap the event field as well f-string call sites interpolate the exception into the message, so event was a third unbounded copy alongside exception and error. * Apply the kwarg-spacing formatter to two files on main * Bound positional args, huge integers and lone surrogates positional_args reach the renderer unformatted, so an exception passed with %s was a fourth unbounded copy. str() on an int past the digit limit raises and JSONResponse cannot encode a lone surrogate, so both turned the 422 into a 500. --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
229 lines
8.3 KiB
Python
229 lines
8.3 KiB
Python
# SPDX-License-Identifier: AGPL-3.0-only
|
|
# Copyright 2026-present the Unsloth AI Inc. team. All rights reserved. See /studio/LICENSE.AGPL-3.0
|
|
|
|
"""A binary body on a JSON route must yield 422, not 500 plus a megabyte of log.
|
|
|
|
POSTing a multipart WAV upload to /api/inference/audio/transcribe (which takes a
|
|
JSON body with base64 audio) failed request validation. The handler then ran
|
|
jsonable_encoder over exc.errors(), whose "input" was the whole raw body, and
|
|
FastAPI encodes bytes with o.decode(): UnicodeDecodeError. The 422 became a 500
|
|
whose traceback embedded the escaped payload, so one 531 KB upload wrote a single
|
|
2.2 MB line into the server log.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import sys
|
|
from pathlib import Path
|
|
|
|
_BACKEND = Path(__file__).resolve().parent.parent
|
|
if str(_BACKEND) not in sys.path:
|
|
sys.path.insert(0, str(_BACKEND))
|
|
|
|
import pytest # noqa: E402
|
|
from fastapi import FastAPI # noqa: E402
|
|
from fastapi.encoders import jsonable_encoder # noqa: E402
|
|
from fastapi.testclient import TestClient # noqa: E402
|
|
from pydantic import BaseModel # noqa: E402
|
|
|
|
from utils.api_errors import ( # noqa: E402
|
|
install_api_error_handlers,
|
|
safe_validation_errors,
|
|
)
|
|
|
|
# Non-UTF-8: 0x80 is an invalid start byte, exactly like a RIFF/WAV payload.
|
|
_BINARY = b"RIFF\x00\x00\x80\xff\xfe\xfd" * 64
|
|
|
|
|
|
class _Body(BaseModel):
|
|
audio: str
|
|
|
|
|
|
def _client() -> TestClient:
|
|
app = FastAPI()
|
|
install_api_error_handlers(app)
|
|
|
|
@app.post("/api/thing")
|
|
async def _thing(payload: _Body): # pragma: no cover - never reached here
|
|
return {"ok": True}
|
|
|
|
return TestClient(app)
|
|
|
|
|
|
def _post_multipart_binary(client: TestClient):
|
|
# A multipart upload to a JSON-body route: the shape that produced the 500.
|
|
# FastAPI reads the body, fails to coerce it to the model, and puts the raw
|
|
# bytes in the error's "input".
|
|
return client.post("/api/thing", files = {"file": ("audio.wav", _BINARY, "audio/wav")})
|
|
|
|
|
|
def test_binary_body_returns_422_not_500():
|
|
resp = _post_multipart_binary(_client())
|
|
assert resp.status_code == 422, resp.text
|
|
|
|
|
|
def test_response_does_not_echo_the_body():
|
|
resp = _post_multipart_binary(_client())
|
|
# The response stays small: the payload is summarized, never echoed back.
|
|
assert len(resp.text) < 2000, len(resp.text)
|
|
assert "RIFF" not in resp.text
|
|
|
|
|
|
def test_valid_body_still_passes():
|
|
resp = _client().post("/api/thing", json = {"audio": "abc"})
|
|
assert resp.status_code == 200
|
|
assert resp.json() == {"ok": True}
|
|
|
|
|
|
def test_a_normal_validation_error_still_names_the_field():
|
|
resp = _client().post("/api/thing", json = {"wrong": 1})
|
|
assert resp.status_code == 422
|
|
body = resp.json()
|
|
detail = body["detail"]
|
|
assert any("audio" in str(item.get("loc", "")) for item in detail), detail
|
|
|
|
|
|
@pytest.mark.parametrize(
|
|
"value, expected_fragment",
|
|
[
|
|
(b"\x80\xff", "2 bytes of binary data"),
|
|
(bytearray(b"\x00" * 5), "5 bytes of binary data"),
|
|
("x" * 5000, "truncated, 5000 chars"),
|
|
],
|
|
)
|
|
def test_summarizer_makes_inputs_encodable(value, expected_fragment):
|
|
errors = [{"type": "x", "loc": ("body",), "msg": "bad", "input": value}]
|
|
safe = safe_validation_errors(errors)
|
|
assert expected_fragment in str(safe[0]["input"])
|
|
jsonable_encoder(safe) # must not raise
|
|
|
|
|
|
def test_short_inputs_are_left_alone():
|
|
errors = [{"type": "x", "loc": ("body",), "msg": "bad", "input": {"a": 1}}]
|
|
assert safe_validation_errors(errors)[0]["input"] == {"a": 1}
|
|
|
|
|
|
def test_nested_binary_inside_a_dict_is_summarized():
|
|
errors = [{"type": "x", "loc": ("body",), "msg": "bad", "input": {"f": b"\x80\x81"}}]
|
|
safe = safe_validation_errors(errors)
|
|
jsonable_encoder(safe)
|
|
assert "2 bytes of binary data" in str(safe[0]["input"]["f"])
|
|
|
|
|
|
def test_a_huge_array_of_small_values_is_bounded():
|
|
# 200k integers are individually tiny but the list was copied whole.
|
|
errors = [{"type": "x", "loc": ("body",), "msg": "bad", "input": list(range(200_000))}]
|
|
safe = safe_validation_errors(errors)
|
|
out = safe[0]["input"]
|
|
assert len(out) <= 21, len(out)
|
|
assert "more items" in str(out[-1])
|
|
assert len(str(safe)) < 2000, len(str(safe))
|
|
|
|
|
|
def test_a_huge_object_of_small_values_is_bounded():
|
|
errors = [
|
|
{"type": "x", "loc": ("body",), "msg": "bad", "input": {str(i): i for i in range(50_000)}}
|
|
]
|
|
out = safe_validation_errors(errors)[0]["input"]
|
|
assert len(out) <= 21, len(out)
|
|
assert "more keys" in str(out["..."])
|
|
|
|
|
|
def test_deep_nesting_does_not_explode():
|
|
value = payload = {}
|
|
for _ in range(20):
|
|
payload["next"] = {}
|
|
payload = payload["next"]
|
|
out = safe_validation_errors([{"type": "x", "loc": ("body",), "msg": "bad", "input": value}])[
|
|
0
|
|
]["input"]
|
|
assert "dict with" in str(out)
|
|
|
|
|
|
def test_a_validator_message_quoting_the_value_is_bounded():
|
|
# models/training.py::_parse_lr raises f"... (got {v!r})".
|
|
msg = "learning_rate must be parseable as float (got '" + "9" * 500_000 + "')"
|
|
safe = safe_validation_errors([{"type": "x", "loc": ("body",), "msg": msg, "input": "x"}])
|
|
assert len(safe[0]["msg"]) < 300, len(safe[0]["msg"])
|
|
assert safe[0]["msg"].startswith("learning_rate must be parseable")
|
|
|
|
|
|
def test_a_ctx_error_quoting_the_value_is_bounded():
|
|
exc = ValueError("got '" + "9" * 500_000 + "'")
|
|
safe = safe_validation_errors(
|
|
[{"type": "x", "loc": ("body",), "msg": "bad", "input": "x", "ctx": {"error": exc}}]
|
|
)
|
|
assert len(str(safe[0]["ctx"]["error"])) < 300
|
|
jsonable_encoder(safe)
|
|
|
|
|
|
def test_a_long_dictionary_key_is_truncated():
|
|
errors = [{"type": "x", "loc": ("body",), "msg": "bad", "input": {"k" * 100_000: 1}}]
|
|
out = safe_validation_errors(errors)[0]["input"]
|
|
assert all(len(k) < 300 for k in out), [len(k) for k in out]
|
|
|
|
|
|
def test_non_finite_numbers_do_not_break_json():
|
|
# Starlette's JSONResponse dumps with allow_nan = False, so an echoed NaN or
|
|
# Infinity turns the intended 422 into a 500.
|
|
import json
|
|
|
|
errors = [
|
|
{"type": "x", "loc": ("body", "max_grad_norm"), "msg": "bad", "input": float("nan")},
|
|
{"type": "x", "loc": ("body", "lr"), "msg": "bad", "input": float("inf")},
|
|
]
|
|
safe = safe_validation_errors(errors)
|
|
json.dumps(jsonable_encoder(safe), allow_nan = False)
|
|
|
|
|
|
def test_the_number_of_errors_is_capped():
|
|
errors = [
|
|
{"type": "x", "loc": ("body", "messages", i, "role"), "msg": "bad", "input": "z"}
|
|
for i in range(5000)
|
|
]
|
|
safe = safe_validation_errors(errors)
|
|
assert len(safe) == 21, len(safe)
|
|
assert "4980 more validation errors omitted" in safe[-1]["msg"]
|
|
|
|
|
|
def test_the_v1_surface_gets_the_same_message_cap():
|
|
from utils.api_errors import _summarize_validation_errors, safe_validation_errors as sve
|
|
|
|
msg = "Unsupported content block type " + "q" * 500_000
|
|
summary, _ = _summarize_validation_errors(
|
|
sve([{"type": "x", "loc": ("body", "messages"), "msg": msg, "input": "x"}])
|
|
)
|
|
assert len(summary) < 400, len(summary)
|
|
|
|
|
|
def test_a_long_loc_element_is_truncated():
|
|
# A typed mapping copies the offending key into loc, so it is user-controlled.
|
|
errors = [{"type": "x", "loc": ("body", "budgets", "k" * 2_000_000), "msg": "bad", "input": 1}]
|
|
out = safe_validation_errors(errors)[0]["loc"]
|
|
assert all(not isinstance(p, str) or len(p) < 300 for p in out), [len(str(p)) for p in out]
|
|
|
|
|
|
def test_an_enormous_integer_is_summarized():
|
|
# str() on an int past sys.get_int_max_str_digits() raises, and json.dumps would
|
|
# otherwise emit every digit.
|
|
import json
|
|
|
|
errors = [{"type": "x", "loc": ("body", "audio"), "msg": "bad", "input": 10**20_000}]
|
|
safe = safe_validation_errors(errors)
|
|
assert "integer with about" in str(safe[0]["input"])
|
|
json.dumps(jsonable_encoder(safe))
|
|
|
|
|
|
def test_an_ordinary_integer_is_left_alone():
|
|
errors = [{"type": "x", "loc": ("body", "batch_size"), "msg": "bad", "input": 4096}]
|
|
assert safe_validation_errors(errors)[0]["input"] == 4096
|
|
|
|
|
|
def test_a_lone_surrogate_can_still_be_encoded():
|
|
# Starlette encodes the response as UTF-8 with ensure_ascii = False, and a lone
|
|
# surrogate survives JSON parsing but cannot be encoded.
|
|
errors = [{"type": "x", "loc": ("body", "batch_size"), "msg": "bad", "input": "\ud800bad"}]
|
|
safe = safe_validation_errors(errors)
|
|
import json
|
|
|
|
json.dumps(jsonable_encoder(safe), ensure_ascii = False).encode("utf-8")
|