[PR #146] feat: add WebHook notifier module #280

New issue

Closed

opened 2026-03-07 19:57:11 +03:00 by kerem · 0 comments

kerem commented

2026-03-07 19:57:11 +03:00

Owner

Original Pull Request: https://github.com/Corsinvest/cv4pve-admin/pull/146

State: closed
Merged: Yes

Summary

Add HookExecutor infrastructure in Core (WebHook, auth types, body types, HTTP methods)
Add WebHook notifier module with configurable URL, method, body template, auth (Basic/Bearer/ApiKey), timeout and SSL options
Rename HttpClient named instances from Proxmox* to generic Http* to support non-PVE HTTP clients
Register HookExecutor service and WebHook module in DI and application startup
Improve label "Validate Certificate" → "Validate SSL Certificate" for clarity

Test plan

Configure a WebHook notifier with POST method and JSON body using %subject% and %body% placeholders
Test Basic, Bearer and ApiKey auth types
Test with IgnoreSslCertificate enabled
Verify existing PVE cluster connections still work after HttpClient rename

**Original Pull Request:** https://github.com/Corsinvest/cv4pve-admin/pull/146 **State:** closed **Merged:** Yes --- ## Summary - Add `HookExecutor` infrastructure in Core (WebHook, auth types, body types, HTTP methods) - Add WebHook notifier module with configurable URL, method, body template, auth (Basic/Bearer/ApiKey), timeout and SSL options - Rename HttpClient named instances from `Proxmox*` to generic `Http*` to support non-PVE HTTP clients - Register `HookExecutor` service and WebHook module in DI and application startup - Improve label "Validate Certificate" → "Validate SSL Certificate" for clarity ## Test plan - [ ] Configure a WebHook notifier with POST method and JSON body using `%subject%` and `%body%` placeholders - [ ] Test Basic, Bearer and ApiKey auth types - [ ] Test with `IgnoreSslCertificate` enabled - [ ] Verify existing PVE cluster connections still work after HttpClient rename