[PR #3139] [MERGED] Added timeout and debugging statuses to webhooks #6144

Closed
opened 2026-02-05 10:25:25 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/3139
Author: @ssddanbrown
Created: 1/3/2022
Status: Merged
Merged: 1/3/2022
Merged by: @ssddanbrown

Base: masterHead: webhook_timeout_and_errors


📝 Commits (2)

  • 00eedaf Added timeout and debugging statuses to webhooks
  • d0de4fd Fixed failing webhook test cases

📊 Changes

14 files changed (+205 additions, -77 deletions)

View changed files

📝 app/Actions/DispatchWebhookJob.php (+14 -5)
📝 app/Actions/Webhook.php (+11 -1)
📝 app/Http/Controllers/WebhookController.php (+2 -0)
📝 database/factories/Actions/WebhookFactory.php (+1 -0)
database/migrations/2022_01_03_154041_add_webhooks_timeout_error_columns.php (+38 -0)
📝 resources/lang/en/common.php (+1 -0)
📝 resources/lang/en/settings.php (+6 -0)
resources/views/form/number.blade.php (+12 -0)
resources/views/settings/roles/form.blade.php (+0 -0)
📝 resources/views/settings/webhooks/create.blade.php (+13 -3)
📝 resources/views/settings/webhooks/edit.blade.php (+40 -4)
📝 resources/views/settings/webhooks/parts/form.blade.php (+51 -62)
📝 tests/Actions/WebhookCallTest.php (+12 -2)
📝 tests/Actions/WebhookManagementTest.php (+4 -0)

📄 Description

  • Added a user-configurable timeout option to webhooks.
  • Added webhook fields for last-call/error datetime, in addition to last
    error string, which are shown on webhook edit view.

Related to #3122


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/3139 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 1/3/2022 **Status:** ✅ Merged **Merged:** 1/3/2022 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `master` ← **Head:** `webhook_timeout_and_errors` --- ### 📝 Commits (2) - [`00eedaf`](https://github.com/BookStackApp/BookStack/commit/00eedafbfdf2e60c7ffebb7e7ccf2a65ffa75b51) Added timeout and debugging statuses to webhooks - [`d0de4fd`](https://github.com/BookStackApp/BookStack/commit/d0de4fd8f9076119e5c06b5ba3887f16ff4b6b64) Fixed failing webhook test cases ### 📊 Changes **14 files changed** (+205 additions, -77 deletions) <details> <summary>View changed files</summary> 📝 `app/Actions/DispatchWebhookJob.php` (+14 -5) 📝 `app/Actions/Webhook.php` (+11 -1) 📝 `app/Http/Controllers/WebhookController.php` (+2 -0) 📝 `database/factories/Actions/WebhookFactory.php` (+1 -0) ➕ `database/migrations/2022_01_03_154041_add_webhooks_timeout_error_columns.php` (+38 -0) 📝 `resources/lang/en/common.php` (+1 -0) 📝 `resources/lang/en/settings.php` (+6 -0) ➕ `resources/views/form/number.blade.php` (+12 -0) ➖ `resources/views/settings/roles/form.blade.php` (+0 -0) 📝 `resources/views/settings/webhooks/create.blade.php` (+13 -3) 📝 `resources/views/settings/webhooks/edit.blade.php` (+40 -4) 📝 `resources/views/settings/webhooks/parts/form.blade.php` (+51 -62) 📝 `tests/Actions/WebhookCallTest.php` (+12 -2) 📝 `tests/Actions/WebhookManagementTest.php` (+4 -0) </details> ### 📄 Description - Added a user-configurable timeout option to webhooks. - Added webhook fields for last-call/error datetime, in addition to last error string, which are shown on webhook edit view. Related to #3122 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 10:25:25 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#6144