From 682c23e0119d42ca35bf67d0967a490ed9aa3b0f Mon Sep 17 00:00:00 2001 From: William Hughes Date: Fri, 6 Sep 2019 11:29:00 +1200 Subject: [PATCH] Treat any 2xx webhook response status code as success --- server/notifications/outgoing.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/notifications/outgoing.js b/server/notifications/outgoing.js index 1dc3d8053..3ee268bbe 100644 --- a/server/notifications/outgoing.js +++ b/server/notifications/outgoing.js @@ -124,7 +124,7 @@ Meteor.methods({ const url = integration.url; const response = postCatchError(url, options); - if (response && response.statusCode && response.statusCode === 200) { + if (response && response.statusCode && response.statusCode >= 200 && response.statusCode < 300) { if (is2way) { const cid = params.commentId; const tooSoon = Lock.has(cid); // if an activity happens to fast, notification shouldn't fire with the same id