fastapi 0.161.0__tar.gz → 0.162.0__tar.gz

{fastapi-0.161.0 → fastapi-0.162.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: fastapi
-Version: 0.161.0
+Version: 0.162.0
 Summary: FastAPI framework, high performance, easy to learn, fast to code, ready for production
 Author-Email: Sebastián Ramírez <tiangolo@gmail.com>
 License-Expression: MIT

fastapi-0.162.0/docs_src/request_forms/tutorial002_an_py310.py

@@ -0,0 +1,10 @@
+from typing import Annotated
+
+from fastapi import FastAPI, Form
+
+app = FastAPI()
+
+
+@app.post("/preferences/")
+async def update_preferences(subscribe: Annotated[bool, Form()] = False):
+    return {"subscribe": subscribe}

fastapi-0.162.0/docs_src/request_forms/tutorial002_py310.py

@@ -0,0 +1,8 @@
+from fastapi import FastAPI, Form
+
+app = FastAPI()
+
+
+@app.post("/preferences/")
+async def update_preferences(subscribe: bool = Form(default=False)):
+    return {"subscribe": subscribe}

{fastapi-0.161.0 → fastapi-0.162.0}/fastapi/__init__.py

@@ -1,6 +1,6 @@
 """FastAPI framework, high performance, easy to learn, fast to code, ready for production"""
 
-__version__ = "0.161.0"
+__version__ = "0.162.0"
 
 from starlette import status as status
 

{fastapi-0.161.0 → fastapi-0.162.0}/fastapi/dependencies/utils.py

@@ -1071,26 +1071,9 @@ def _extract_form_items(
     field_aliases: set[str] = set()
 
     for field in body_fields:
-        validation_alias = field.field_info.validation_alias
-        has_alias_path = isinstance(validation_alias, AliasPath) or (
-            isinstance(validation_alias, AliasChoices)
-            and any(
-                isinstance(choice, AliasPath) for choice in validation_alias.choices
-            )
-        )
-        aliases = [alias for alias, _ in _get_param_aliases(field)]
-        # Interpret missing boolean form fields as unchecked HTML checkboxes.
-        infer_false_if_missing = (
-            not has_alias_path
-            and field.field_info.annotation in (bool, bool | None)
-            and not any(alias in received_body for alias in aliases)
-        )
-        for alias in aliases:
+        for alias, _ in _get_param_aliases(field):
             field_aliases.add(alias)
-            if infer_false_if_missing:
-                value: Any = False
-            else:
-                value = _get_multidict_value(field, received_body, alias=alias)
+            value = _get_multidict_value(field, received_body, alias=alias)
             if value is not Undefined:
                 yield alias, value, field
                 # Leave later aliases unchanged for other fields or extra validation.
@@ -1141,7 +1124,7 @@ def _is_form_model_supplied(
     model: type[BaseModel],
     received_body: FormData,
 ) -> bool:
-    # Check raw keys before empty fields are removed or checkboxes inferred.
+    # Check raw keys before empty fields are removed.
     own_names = {
         alias
         for item in get_cached_model_fields(model)
@@ -1366,8 +1349,7 @@ def _get_body_field(
         return None
     first_param = body_params[0]
     has_nullable_form_model = any(
-        (model := _get_form_model(field)) is not None
-        and field.field_info.annotation is not model
+        _get_form_models(field) and type(None) in get_args(field.field_info.annotation)
         for field in body_params
     )
     if not embed_body_fields and not has_nullable_form_model:
@@ -1402,6 +1384,9 @@ def _get_body_field(
     if not embed_body_fields:
         field_info = copy(first_param.field_info)
         field_info.annotation = BodyModel
+        # The original form field already carries the validation metadata.
+        field_info.metadata = []
+        field_info.discriminator = None
         return ModelField(name=first_param.name, field_info=field_info)
     required = any(True for f in body_params if f.field_info.is_required())
     BodyFieldInfo_kwargs: dict[str, Any] = {

{fastapi-0.161.0 → fastapi-0.162.0}/pyproject.toml

@@ -53,7 +53,7 @@ dependencies = [
     "annotated-doc>=0.0.2",
     "opentelemetry-api>=1.44.0",
 ]
-version = "0.161.0"
+version = "0.162.0"
 
 [project.urls]
 Homepage = "https://github.com/fastapi/fastapi"

{fastapi-0.161.0 → fastapi-0.162.0}/tests/test_forms_multiple_models.py

@@ -232,7 +232,7 @@ def test_composed_form_models_forbid_extras(multipart, path):
     assert models["Preferences"]["additionalProperties"] is False
 
 
-def test_model_defaults_checkboxes_and_validators_are_independent():
+def test_model_defaults_and_validators_are_independent():
     app = FastAPI()
     inputs = []
 
@@ -274,9 +274,9 @@ def test_model_defaults_checkboxes_and_validators_are_independent():
     response = client.post("/")
     assert response.status_code == 200, response.text
     assert response.json() == {
-        "settings": {"label": "default", "enabled": False, "tags": []},
+        "settings": {"label": "default", "enabled": True, "tags": []},
         "other": {"label": "other", "enabled": "other"},
-        "fields_set": ["enabled"],
+        "fields_set": [],
         "other_fields_set": [],
     }
     response = client.post(

{fastapi-0.161.0 → fastapi-0.162.0}/tests/test_forms_multiple_unions.py

@@ -18,7 +18,7 @@ class Person(BaseModel):
     kind: Literal["person"] = Field(alias="contact_type")
     name: str
     tags: list[str] = Field(default_factory=list)
-    enabled: bool = True
+    enabled: bool = False
 
 
 class Company(BaseModel):
@@ -704,7 +704,7 @@ def test_json_and_original_model_schemas_are_unchanged():
         json={"contact": {"contact_type": "person", "name": "Rick"}, "token": "secret"},
     )
     assert response.status_code == 200, response.text
-    assert response.json()["enabled"] is True
+    assert response.json()["enabled"] is False
     schemas = app.openapi()["components"]["schemas"]
     assert set(schemas["Person"]["properties"]) == {
         "contact_type",

{fastapi-0.161.0 → fastapi-0.162.0}/tests/test_forms_optional_models.py

@@ -22,7 +22,7 @@ def test_optional_form_model():
         name: str
         age: int
         tags: list[str] = Field(default_factory=list)
-        enabled: bool = True
+        enabled: bool = False
 
     @app.post("/")
     def submit(profile: Annotated[Profile | None, Form()] = None):
@@ -114,7 +114,7 @@ def test_optional_form_model_multipart():
         name: str
         age: int
         tags: list[str] = Field(default_factory=list)
-        enabled: bool = True
+        enabled: bool = False
 
     @app.post("/")
     def submit(
@@ -176,7 +176,7 @@ def test_optional_form_model_schema(extra):
         name: str
         age: int
         tags: list[str] = Field(default_factory=list)
-        enabled: bool = True
+        enabled: bool = False
 
     @app.post("/")
     def submit(profile: Annotated[Profile | None, Form()] = None):
@@ -224,7 +224,7 @@ def test_optional_form_model_with_other_form_field_schema(extra):
         name: str
         age: int
         tags: list[str] = Field(default_factory=list)
-        enabled: bool = True
+        enabled: bool = False
 
     @app.post("/")
     def submit(
@@ -276,7 +276,7 @@ def test_unknown_fields_and_sibling_models(extra, required_name):
     class Profile(BaseModel):
         model_config = ConfigDict(extra=extra)
         name: str = Field() if required_name else Field("default")
-        enabled: bool = True
+        enabled: bool = False
 
     class Credentials(BaseModel):
         token: str
@@ -458,7 +458,7 @@ def test_defaults_factories_and_validators():
     calls.clear()
     response = client.post("/factory", data={"name": "submitted"})
     assert response.status_code == 200, response.text
-    assert response.json()["enabled"] is False
+    assert response.json()["enabled"] is True
     assert calls == ["submitted"]
     calls.clear()
     response = client.post("/validated-default")

{fastapi-0.161.0 → fastapi-0.162.0}/tests/test_forms_optional_unions.py

@@ -19,7 +19,7 @@ app = FastAPI()
 class TextForm(BaseModel):
     kind: Literal["text"]
     value: str
-    enabled: bool = True
+    enabled: bool = False
 
 
 class ListForm(BaseModel):
@@ -125,15 +125,27 @@ def test_nullable_union_without_default_is_required():
         return data
 
     client = TestClient(app)
-    response = client.post("/")
-    assert response.status_code == 422
-    assert response.json()["detail"] == [
-        {"type": "missing", "loc": ["body"], "msg": "Field required", "input": None}
-    ]
+    for fields in ({}, {"unknown": "ignored"}):
+        response = client.post("/", data=fields)
+        assert response.status_code == 422
+        assert response.json()["detail"] == [
+            {"type": "missing", "loc": ["body"], "msg": "Field required", "input": None}
+        ]
     response = client.post("/", data={"count": "2"})
     assert response.status_code == 200, response.text
     assert response.json() == {"count": 2}
-    assert app.openapi()["paths"]["/"]["post"]["requestBody"]["required"] is True
+    schema = app.openapi()
+    assert schema["paths"]["/"]["post"]["requestBody"]["required"] is True
+    assert schema["components"]["schemas"]["Body_read__post"]["allOf"] == [
+        {"not": {"not": {"anyOf": [{"required": ["count"]}, {"required": ["label"]}]}}},
+        {
+            "anyOf": [
+                {"$ref": "#/components/schemas/Count"},
+                {"$ref": "#/components/schemas/Label"},
+            ],
+            "title": "Data",
+        },
+    ]
 
 
 @pytest.mark.parametrize("extra", ["ignore", "allow", "forbid"])
@@ -173,6 +185,27 @@ def test_unknown_fields(extra):
         assert response.json() == (
             {"count": 1, "unknown": ["a", "b"]} if extra == "allow" else None
         )
+    schema = app.openapi()
+    assert "required" not in schema["paths"]["/"]["post"]["requestBody"]
+    absent = (
+        {"maxProperties": 0}
+        if extra in ("allow", "forbid")
+        else {"not": {"anyOf": [{"required": ["count"]}, {"required": ["label"]}]}}
+    )
+    assert schema["components"]["schemas"]["Body_read__post"]["allOf"] == [
+        {
+            "anyOf": [
+                absent,
+                {
+                    "anyOf": [
+                        {"$ref": "#/components/schemas/Count"},
+                        {"$ref": "#/components/schemas/Label"},
+                    ],
+                    "title": "Data",
+                },
+            ]
+        }
+    ]
 
 
 def test_mixed_extra_configurations_activate_union():
@@ -339,12 +372,16 @@ def test_defaults_factories_and_validators():
     calls.clear()
     response = client.post("/factory", data={"kind": "defaults"})
     assert response.status_code == 200, response.text
-    assert response.json() == {"kind": "defaults", "tags": [], "enabled": False}
+    assert response.json() == {"kind": "defaults", "tags": [], "enabled": True}
     assert calls == ["validated"]
     calls.clear()
     response = client.post("/validated-default")
     assert response.status_code == 200, response.text
     assert calls == ["validated"]
+    schema = app.openapi()
+    for path in ("/default", "/factory", "/validated-default"):
+        assert "required" not in schema["paths"][path]["post"]["requestBody"]
+    assert calls == ["validated"]
 
 
 def test_optional_union_in_dependency():
@@ -370,24 +407,51 @@ def test_optional_union_in_dependency():
 
 def test_openapi():
     schema = app.openapi()
+    schemas = schema["components"]["schemas"]
+    absent = {
+        "not": {
+            "anyOf": [
+                {"required": ["enabled"]},
+                {"required": ["kind"]},
+                {"required": ["value"]},
+            ]
+        }
+    }
     assert schema["paths"]["/union"]["post"]["requestBody"] == {
         "content": {
             "application/x-www-form-urlencoded": {
-                "schema": {
+                "schema": {"$ref": "#/components/schemas/Body_read_union_union_post"}
+            }
+        }
+    }
+    assert schemas["Body_read_union_union_post"]["allOf"] == [
+        {
+            "anyOf": [
+                absent,
+                {
                     "anyOf": [
                         {"$ref": "#/components/schemas/TextForm"},
                         {"$ref": "#/components/schemas/ListForm"},
-                        {"type": "null"},
                     ],
                     "title": "Data",
-                }
-            }
+                },
+            ]
         }
-    }
+    ]
     assert schema["paths"]["/discriminated"]["post"]["requestBody"] == {
         "content": {
             "application/x-www-form-urlencoded": {
                 "schema": {
+                    "$ref": "#/components/schemas/Body_read_discriminated_discriminated_post"
+                }
+            }
+        }
+    }
+    assert schemas["Body_read_discriminated_discriminated_post"]["allOf"] == [
+        {
+            "anyOf": [
+                absent,
+                {
                     "anyOf": [
                         {
                             "oneOf": [
@@ -402,12 +466,11 @@ def test_openapi():
                                 },
                             },
                         },
-                        {"type": "null"},
                     ],
                     "title": "Data",
-                }
-            }
+                },
+            ]
         }
-    }
-    assert schema["components"]["schemas"]["TextForm"]["required"] == ["kind", "value"]
-    assert schema["components"]["schemas"]["ListForm"]["required"] == ["kind", "value"]
+    ]
+    assert schemas["TextForm"] == TextForm.model_json_schema()
+    assert schemas["ListForm"] == ListForm.model_json_schema()