Treat any 2xx webhook response status code as success

6 years ago · 682c23e011
parent d4a307f66d
commit 682c23e011
1 changed files with 1 additions and 1 deletions
--- 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