Azure / Azure/azure-functions-durable-python

Custom object serialization broken for orchestrator return values

オープン 初心者向け
#568 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
bug fixed-in-v2 P2
主要言語
Python
スター
157
フォーク
70
平均マージ
2日 10時間
マージ済み PR(30日)
2

説明

🐛 **Describe the bug**
Azure Durable Functions Python SDK fails to serialize custom objects that implement a `to_json()` method when returning from orchestrator functions. The error "_Object of type [CustomClass] is not JSON serializable_" is raised.

🤔 **Expected behavior**
Custom objects implementing a `to_json()` method should be automatically serialized when returned from orchestrator functions.

☕ **Steps to reproduce**
Basic orchestration with custom return types.

1. Create a custom class with `to_json()` and `from_json()` methods:
```python
from dataclasses import dataclass

import azure.durable_functions as df
import azure.functions as func

app = df.DFApp(http_auth_level=func.AuthLevel.ANONYMOUS)

@dataclass
class CustomResult():
message: str
success: bool = True

def to_json(self):
return {"message": self.message, "success": self.success}

@classmethod
def from_json(cls, data):
return cls(message=data["message"], success=data["success"])

@app.orchestration_trigger(context_name="context")
def orchestrator_function(context: df.DurableOrchestrationContext):
return CustomResult(message="Hello from orchestrator!", success=True)

@app.route(route="test", methods=["GET"])
@app.durable_client_input(client_name="client")
async def http_start_conversion(req: func.HttpRequest, client: df.DurableOrchestrationClient) -> func.HttpResponse:
instance_id = await client.start_new(
orchestration_function_name="orchestrator_function"
)

return client.create_check_status_response(req, instance_id)

```

2. Execute the orchestration - it will fail with "Object of type CustomResult is not JSON serializable"

**Environment**: Running locally with
Azure Functions Core Tools Version: 4.1.0
azure-functions-durable: 1.3.2

**Root cause**: In `TaskOrchestrationExecutor.get_orchestrator_state_str()`, the code calls dumps without a default argument:
https://github.com/Azure/azure-functions-durable-python/blob/15ed958555b9410629f97cd7884d8783cb0e460e/azure/durable_functions/models/TaskOrchestrationExecutor.py#L284

But should call:
`json.dumps(self._data, default=_serialize_custom_object)` as in other models or entities.

⚡**If deployed to Azure**
Didn't test yet. Since the bug is in the Python SDK code itself, I expect the same issue to occur.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

azure/durable_functions/models/TaskOrchestrationExecutor.py の TaskOrchestrationExecutor.get_orchestrator_state_str() から始め、その JSON シリアライゼーションを issue で言及されている他のモデルまたはエンティティと比較します。CustomResult の例を使ってオーケストレーションを再現し、to_json() を持つオブジェクトを not-serializable エラーなしで返せることを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
python
領域
backend
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
静か
明瞭さ
明確に書かれている
初心者へのやさしさ
75/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。