Gandalf/sofarr
1
0
Fork 0
Code Issues 1 Pull Requests Actions 1 Packages Projects Releases 22 Wiki Activity

fix(webhooks): Send full notification object to test endpoint
All checks were successful
Build and Push Docker Image / build (push) Successful in 48s
Details
Licence Check / Licence compatibility and copyright header verification (push) Successful in 1m9s
Details
CI / Security audit (push) Successful in 1m23s
Details
CI / Tests & coverage (push) Successful in 1m42s
Details

Browse Source 
The /notifications/test endpoint requires the full notification object,
not just the ID. Changed testSonarrWebhook() and testRadarrWebhook() to
send the complete notification object (sonarrSofarr/radarrSofarr).

Fixes: 400 validation error when testing webhooks
This commit is contained in:
Gronod
2026-05-19 21:16:31 +01:00
parent 0f3c02e52d
commit 3ef35a8c43
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
public/app.js
View File

@@ -1396,7 +1396,7 @@ async function testSonarrWebhook() {
'Content-Type': 'application/json', 'Content-Type': 'application/json',
'X-CSRF-Token': csrfToken || '' 'X-CSRF-Token': csrfToken || ''
}, },
body: JSON.stringify({ id: sonarrSofarr.id }) body: JSON.stringify(sonarrSofarr)
}); });
if (!res.ok) throw new Error('Test failed'); if (!res.ok) throw new Error('Test failed');
await fetchWebhookStatus(); await fetchWebhookStatus();
@@ -1424,7 +1424,7 @@ async function testRadarrWebhook() {
'Content-Type': 'application/json', 'Content-Type': 'application/json',
'X-CSRF-Token': csrfToken || '' 'X-CSRF-Token': csrfToken || ''
}, },
body: JSON.stringify({ id: radarrSofarr.id }) body: JSON.stringify(radarrSofarr)
}); });
if (!res.ok) throw new Error('Test failed'); if (!res.ok) throw new Error('Test failed');
await fetchWebhookStatus(); await fetchWebhookStatus();