diff options
| author | Lauri Ojansivu <x@xet7.org> | 2019-09-06 19:22:36 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-09-06 19:22:36 +0300 |
| commit | ea26f706691fde08a397c69ca67309bda43046fa (patch) | |
| tree | f67969135ae5b5394921be32f1858e713913ff52 | |
| parent | d4a307f66d254e6eb832fc6397c802b02b4dacee (diff) | |
| parent | 682c23e0119d42ca35bf67d0967a490ed9aa3b0f (diff) | |
| download | wekan-ea26f706691fde08a397c69ca67309bda43046fa.tar.gz wekan-ea26f706691fde08a397c69ca67309bda43046fa.tar.bz2 wekan-ea26f706691fde08a397c69ca67309bda43046fa.zip | |
Merge pull request #2683 from insertjokehere/fix_error_on_successful_webhook
Treat any 2xx webhook response status code as success
| -rw-r--r-- | server/notifications/outgoing.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/server/notifications/outgoing.js b/server/notifications/outgoing.js index 1dc3d805..3ee268bb 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 |