fix(webhooks): Use numeric method value (1=POST) in notification payload
All checks were successful
Build and Push Docker Image / build (push) Successful in 44s
Licence Check / Licence compatibility and copyright header verification (push) Successful in 1m6s
CI / Security audit (push) Successful in 1m17s
CI / Tests & coverage (push) Successful in 1m33s

The webhook notification payload was using string 'POST' for the method
field, but Sonarr/Radarr API expects numeric values:
- 1 = POST
- 2 = PUT

Also added onManualInteractionRequired: false to match the schema.

Fixes: Radarr/Sonarr rejecting webhook configuration with validation errors
This commit is contained in:
2026-05-19 20:47:19 +01:00
parent 9fd60bcfed
commit 0f3c02e52d
2 changed files with 8 additions and 6 deletions

View File

@@ -196,16 +196,17 @@ router.post('/notifications/sofarr-webhook', async (req, res) => {
configContract: 'WebhookSettings',
fields: [
{ name: 'url', value: webhookUrl },
{ name: 'method', value: 'POST' },
{ name: 'method', value: 1 },
{ name: 'headers', value: [{ key: 'X-Sofarr-Webhook-Secret', value: webhookSecret }] }
],
onGrab: true,
onDownload: true,
onImport: true,
onUpgrade: true,
onImport: true,
onRename: false,
onHealthIssue: false,
onApplicationUpdate: false
onApplicationUpdate: false,
onManualInteractionRequired: false
};
if (existingNotification) {

View File

@@ -196,16 +196,17 @@ router.post('/notifications/sofarr-webhook', async (req, res) => {
configContract: 'WebhookSettings',
fields: [
{ name: 'url', value: webhookUrl },
{ name: 'method', value: 'POST' },
{ name: 'method', value: 1 },
{ name: 'headers', value: [{ key: 'X-Sofarr-Webhook-Secret', value: webhookSecret }] }
],
onGrab: true,
onDownload: true,
onImport: true,
onUpgrade: true,
onImport: true,
onRename: false,
onHealthIssue: false,
onApplicationUpdate: false
onApplicationUpdate: false,
onManualInteractionRequired: false
};
if (existingNotification) {