kiba-core 0.5.3.dev74__tar.gz → 0.5.3.dev75__tar.gz

{kiba_core-0.5.3.dev74 → kiba_core-0.5.3.dev75}/CHANGELOG.md

@@ -8,6 +8,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 ## [Unreleased]
 
 ### Added
+- [PATCH] Fixed `json_route` and `streaming_json_route` returning a 500 when the JSON body isn't an object (e.g. `true` or `[1]`); they now raise `BadRequestException` (400)
 - [MINOR] Added `shouldNotifyRetriedFailures` (default `True`) to `MessageQueueProcessor`: when `False`, failure notifications are only sent once the queue gives up on a message, not for failures it will retry. `MessageQueue.fail_message` now returns whether the queue scheduled another attempt: `SqlMessageQueue` and `CosmosMessageQueue` return `True` until `maxAttempts` is reached, while `SqsMessageQueue` and `AqsMessageQueue` always return `False` as the platform decides on redelivery, so they keep notifying every failure
 - [MAJOR] Updated `CosmosMessageQueue` to retry failed messages like `SqlMessageQueue`: added `maxAttempts` (default 3) and `failureRetryDelaySeconds` (default 60). `fail_message` reschedules the message with linear backoff and stores `lastError`, and a visible message whose lease expired counts as an attempt when it is claimed again. Once `postCount` reaches `maxAttempts` the message is removed, along with its deduplication item. Previously failed or abandoned messages reappeared once their lease expired and were retried forever
 - [MINOR] Added `pollLogIntervalSeconds` (default 600) to `MessageQueueProcessor`: `Retrieving messages...` is now logged on the first poll and then at most once per interval instead of on every poll, so it still shows the worker is alive without flooding the logs; pass `0` to log every poll as before

{kiba_core-0.5.3.dev74 → kiba_core-0.5.3.dev75}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kiba-core
-Version: 0.5.3.dev74
+Version: 0.5.3.dev75
 Summary: Kiba Labs' python utilities
 Project-URL: repository, https://github.com/kibalabs/core-py
 Requires-Python: <4,>=3.12

{kiba_core-0.5.3.dev74 → kiba_core-0.5.3.dev75}/core/api/json_route.py

@@ -30,6 +30,8 @@ def json_route[ApiRequest: BaseModel, ApiResponse: BaseModel](
                     body = typing.cast(JsonObject, json_util.loads(bodyBytes.decode()))
                 except json_util.JsonDecodeException as exception:
                     raise BadRequestException(f'Invalid JSON body: {exception}')
+                if not isinstance(body, dict):
+                    raise BadRequestException('Invalid JSON body: expected an object')
             allParams = {**pathParams, **body, **queryParams}
             try:
                 requestParams = requestType(**allParams)

{kiba_core-0.5.3.dev74 → kiba_core-0.5.3.dev75}/core/api/streaming_json_route.py

@@ -39,6 +39,8 @@ def streaming_json_route[ApiRequest: BaseModel, ApiResponse: BaseModel](
                     body = typing.cast(JsonObject, json_util.loads(bodyBytes.decode()))
                 except json_util.JsonDecodeException as exception:
                     raise BadRequestException(f'Invalid JSON body: {exception}')
+                if not isinstance(body, dict):
+                    raise BadRequestException('Invalid JSON body: expected an object')
             allParams = {**pathParams, **body, **queryParams}
             try:
                 requestParams = requestType(**allParams)

{kiba_core-0.5.3.dev74 → kiba_core-0.5.3.dev75}/kiba_core.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kiba-core
-Version: 0.5.3.dev74
+Version: 0.5.3.dev75
 Summary: Kiba Labs' python utilities
 Project-URL: repository, https://github.com/kibalabs/core-py
 Requires-Python: <4,>=3.12

{kiba_core-0.5.3.dev74 → kiba_core-0.5.3.dev75}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "kiba-core"
-version = "0.5.3.dev74"
+version = "0.5.3.dev75"
 description = "Kiba Labs' python utilities"
 readme = "README.md"
 requires-python = ">=3.12, <4"

{kiba_core-0.5.3.dev74 → kiba_core-0.5.3.dev75}/tests/api/test_json_route.py

@@ -134,3 +134,14 @@ def test_json_route_empty_body(client):
     data = response.json()
     assert data["request_name"] == "EmptyBody"
     assert data["request_age"] == 50
+
+
+@pytest.mark.parametrize("content", ["true", "[1, 2]", "\"text\"", "null", "1"])
+def test_json_route_with_non_object_json(client, content):
+    response = client.post(
+        "/test?name=NonObject&age=1",
+        content=content,
+        headers={"Content-Type": "application/json"}
+    )
+    assert response.status_code == 400
+    assert "expected an object" in response.json().get("message", "")

{kiba_core-0.5.3.dev74 → kiba_core-0.5.3.dev75}/tests/api/test_streaming_json_route.py

@@ -184,3 +184,14 @@ def test_error_thrown_by_endpoint(client):
     data = dataList[0]
     assert data["request_name"] == "Name"
     assert data["request_age"] == 30
+
+
+@pytest.mark.parametrize("content", ["true", "[1, 2]", "\"text\"", "null", "1"])
+def test_streaming_json_route_with_non_object_json(client, content):
+    response = client.post(
+        "/test?name=NonObject&age=1",
+        content=content,
+        headers={"Content-Type": "application/json"}
+    )
+    assert response.status_code == 400
+    assert "expected an object" in response.json().get("message", "")