Skip to content

Commit

Permalink
🔄 synced local 'skyvern/' with remote 'skyvern/'
Browse files Browse the repository at this point in the history
<!-- ELLIPSIS_HIDDEN -->

| 🚀 | This description was created by [Ellipsis](https://www.ellipsis.dev) for commit ddeae74ac4ac0364f2c99917a810ca8648fbd335  |
|--------|--------|

### Summary:
Removed JSON response logging in `send_workflow_response`, affecting webhook response logging behavior.

**Key points**:
- Removed JSON response logging in `send_workflow_response` function.
- Affects `skyvern/forge/sdk/workflow/service.py`.
- Removed `resp_json=resp.json()` from logging when webhook fails.
- Changes logging behavior for webhook responses.

----
Generated with ❤️ by [ellipsis.dev](https://www.ellipsis.dev)

<!-- ELLIPSIS_HIDDEN -->
  • Loading branch information
ykeremy committed Aug 8, 2024
1 parent 5299f92 commit d5e9dd6
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion skyvern/forge/sdk/workflow/service.py
Original file line number Diff line number Diff line change
Expand Up @@ -729,10 +729,10 @@ async def send_workflow_response(
"Webhook failed",
workflow_id=workflow.workflow_id,
workflow_run_id=workflow_run.workflow_run_id,
webhook_data=payload,
resp=resp,
resp_code=resp.status_code,
resp_text=resp.text,
resp_json=resp.json(),
)
except Exception as e:
raise FailedToSendWebhook(
Expand Down

0 comments on commit d5e9dd6

Please sign in to comment.