harmacist/healthchecks
1
0
Fork 0
forked from GithubBackups/healthchecks
Code Issues Pull Requests Projects Releases Wiki Activity

In api.views.notification_status, always return HTTP 200 so the other party doesn't retry over and over again

Browse Source
This commit is contained in:
Pēteris Caune 2020-08-17 13:10:07 +03:00
parent 697cb19bde
commit a29b82a0ed
No known key found for this signature in database
GPG Key ID: E28D7679E9A9EDE2
2 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
hc/api/tests/test_notification_status.py
View File

@ -57,7 +57,11 @@ class NotificationStatusTestCase(BaseTestCase):
self.n.save()
r = self.client.post(self.url, {"MessageStatus": "failed"})
self.assertEqual(r.status_code, 403)
self.assertEqual(r.status_code, 200)
# The notification should not have the error field set:
self.n.refresh_from_db()
self.assertEqual(self.n.error, "")
def test_it_handles_missing_notification(self):
fake_code = "07c2f548-9850-4b27-af5d-6c9dc157ec02"

5
hc/api/views.py
View File

@ -422,10 +422,11 @@ def notification_status(request, code):
notification = get_object_or_404(Notification, code=code)
# If webhook is more than 1 hour late, don't accept it:
td = timezone.now() - notification.created
if td.total_seconds() > 3600:
return HttpResponseForbidden()
# If the webhook is called more than 1 hour after the notification, ignore it.
# Return HTTP 200 so the other party doesn't retry over and over again:
return HttpResponse()
error, mark_not_verified = None, False