From 0f3c02e52d1ea5783814f77e0c05f3b84c903f69 Mon Sep 17 00:00:00 2001
From: Gronod <gordon@i3omb.com>
Date: Tue, 19 May 2026 20:47:19 +0100
Subject: [PATCH] fix(webhooks): Use numeric method value (1=POST) in
 notification payload

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
---
 server/routes/radarr.js | 7 ++++---
 server/routes/sonarr.js | 7 ++++---
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/server/routes/radarr.js b/server/routes/radarr.js
index eceb866..754b4c6 100644
--- a/server/routes/radarr.js
+++ b/server/routes/radarr.js
@@ -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) {
diff --git a/server/routes/sonarr.js b/server/routes/sonarr.js
index 0704cb6..71661f1 100644
--- a/server/routes/sonarr.js
+++ b/server/routes/sonarr.js
@@ -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) {