batch response bodies are dropped when they are JSON objects

Đang mở
#737 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
72/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
api, backend

Hướng nghiên cứu

Bắt đầu với bộ giải tuần tự body tại src/msgraph_core/requests/batch_response_item.py, sau đó kiểm tra JsonParseNode._get_bytes_value() trong kiota serialization JSON package và luồng BatchResponseContent. Tái hiện một phản hồi batch có body riêng lẻ là một đối tượng JSON thông qua BatchRequestBuilder; hoàn tất khi đối tượng được giữ lại trong BatchResponseItem.body thay vì trở thành None.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Summary

After upgrading to the Kiota Python version containing #646, batch responses from Microsoft Graph can no longer be deserialized correctly when an individual response body is a JSON object.

I am updating SharePoint list items in a batch request. The HTTP adapter returns the batch response body as a Python dict. During deserialization, BatchResponseItem passes that value to JsonParseNode.get_bytes_value(). The current _get_bytes_value() implementation only handles str values and returns None for dictionaries, so the response body is silently swallowed.

Before #646, non-string values were converted to a JSON string/UTF-8 byte sequence. That byte sequence was later parsed again as JSON, restoring the response body as a dictionary. #646 removed the non-string fallback needed for JSON object bodies.

This issue might originally stem from incorrect use of JsonParseNode in https://github.com/microsoftgraph/msgraph-sdk-python-core/blob/d23507bf38497aa514763f3ffa56b8ed8eceef70/src/msgraph_core/requests/batch_response_item.py#L159 but it didn't occur until the changes made in #646. If a fix should rather be made in msgraph-core, please let me know and I will raise the issue there.

Environment

  • Python: 3.13
  • microsoft-kiota-bundle: 1.12.3
  • microsoft-kiota-serialization-json: 1.12.3
  • microsoft-kiota-http: 1.12.3
  • msgraph-core: 1.5.1

Reproduction

  1. Create a Microsoft Graph batch request containing SharePoint list-item updates.
  2. Send the request through msgraph_core.requests.batch_request_builder.BatchRequestBuilder.
  3. Return a successful response whose individual body is a JSON object.
  4. Deserialize the batch response as BatchResponseContent.

The individual BatchResponseItem.body is expected to contain the JSON object, but it becomes None during deserialization.

Debugger call stack

I'm including a trace to where my app is hitting this regression.

Traceback (most recent call last):
  File ".venv/Lib/site-packages/msgraph_core/requests/batch_request_builder.py", line 103, in _post_batch_collection
    response = await self.post(batch_request_content, error_map)
  File ".venv/Lib/site-packages/msgraph_core/requests/batch_request_builder.py", line 65, in post
    response = await self._request_adapter.send_async(
  File ".venv/Lib/site-packages/kiota_http/httpx_request_adapter.py", line 198, in send_async
    value = root_node.get_object_value(parsable_factory)
  File ".venv/Lib/site-packages/kiota_serialization_json/json_parse_node.py", line 213, in get_object_value
    self._assign_field_values(result)
  File ".venv/Lib/site-packages/kiota_serialization_json/json_parse_node.py", line 352, in _assign_field_values
    field_deserializer(JsonParseNode(field_value))
  File ".venv/Lib/site-packages/msgraph_core/requests/batch_response_content.py", line 156, in <lambda>
    return {'responses': lambda n: set_responses(n)}
  File ".venv/Lib/site-packages/msgraph_core/requests/batch_response_content.py", line 150, in set_responses
    values = n.get_collection_of_object_values(BatchResponseItem)
  File ".venv/Lib/site-packages/kiota_serialization_json/json_parse_node.py", line 153, in <lambda>
    lambda x: self._create_new_node(x).get_object_value(factory),
  File ".venv/Lib/site-packages/kiota_serialization_json/json_parse_node.py", line 213, in get_object_value
    self._assign_field_values(result)
  File ".venv/Lib/site-packages/kiota_serialization_json/json_parse_node.py", line 352, in _assign_field_values
    field_deserializer(JsonParseNode(field_value))
  File ".venv/Lib/site-packages/msgraph_core/requests/batch_response_item.py", line 159, in <lambda>
    "body": lambda x: setattr(self, "body", x.get_bytes_value()),
  File ".venv/Lib/site-packages/kiota_serialization_json/json_parse_node.py", line 223, in get_bytes_value
    return self._get_bytes_value(self._json_node)
  File ".venv/Lib/site-packages/kiota_serialization_json/json_parse_node.py", line 296, in _get_bytes_value
    return None
Ngôn ngữ chính
Python
Star
31
Fork
39
Merge trung bình
11 giờ 45 phút
Pull request đã merge (30 ngày)
42

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của microsoft/kiota-python

Tất cả issue của microsoft/kiota-python

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.