Skip to content

Chore: add webhook logging#2647

Merged
alexjustesen merged 6 commits intoalexjustesen:mainfrom
svenvg93:chore/webhook-loggign
Feb 3, 2026
Merged

Chore: add webhook logging#2647
alexjustesen merged 6 commits intoalexjustesen:mainfrom
svenvg93:chore/webhook-loggign

Conversation

@svenvg93
Copy link
Contributor

@svenvg93 svenvg93 commented Jan 11, 2026

📃 Description

The webhook notification were fire and forget. There is was no feedback to the user nor logging when it went wrong.

🪵 Changelog

➕ Added

  • Add LogWebhookFailure listener that listens for WebhookCallFailedEvent and creates a log.
    • [2026-01-11 16:42:58] local.ERROR: Webhook notification failed {"url":"http://192.168.1.5:8001/webhook","attempt":1,"http_verb":"post","error_type":"GuzzleHttp\\Exception\\ConnectException","error_message":"cURL error 28: Operation timed out after 3002 milliseconds with 0 bytes received (see https://curl.haxx.se/libcurl/c/libcurl-errors.html) for http://192.168.1.5:8001/webhook"}

@svenvg93 svenvg93 requested a review from alexjustesen January 25, 2026 16:46
@svenvg93 svenvg93 changed the title chore: add webhook logging Chore: add webhook logging Jan 27, 2026
@alexjustesen alexjustesen merged commit 3fa025d into alexjustesen:main Feb 3, 2026
9 checks passed
@svenvg93 svenvg93 deleted the chore/webhook-loggign branch February 3, 2026 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants