Merge pull request #697 from pennlabs/pca-quick-fix

Return 200 to webhook request
pennlabs · Jan 17, 2025 · cb694bf · cb694bf
2 parents 5c6f189 + 94c0ae5
commit cb694bf
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/backend/alert/views.py b/backend/alert/views.py
@@ -66,7 +66,7 @@ def accept_webhook(request):
             """Your credentials cannot be verified.
         They should be placed in the header as &quot;Authorization-Bearer&quot;,
         YOUR_APP_ID and &quot;Authorization-Token&quot; , YOUR_TOKEN""",
-            status=401,
+            status=200,
         )
 
     if request.method != "POST":

diff --git a/backend/tests/alert/test_alert.py b/backend/tests/alert/test_alert.py
@@ -912,7 +912,7 @@ def test_wrong_password(self, mock_alert):
             content_type="application/json",
             **self.headers,
         )
-        self.assertEqual(401, res.status_code)
+        self.assertEqual(200, res.status_code)
         self.assertFalse(mock_alert.called)
         self.assertEqual(0, StatusUpdate.objects.count())
 
@@ -926,7 +926,7 @@ def test_wrong_user(self, mock_alert):
             content_type="application/json",
             **self.headers,
         )
-        self.assertEqual(401, res.status_code)
+        self.assertEqual(200, res.status_code)
         self.assertFalse(mock_alert.called)
         self.assertEqual(0, StatusUpdate.objects.count())
 

diff --git a/k8s/main.ts b/k8s/main.ts
@@ -141,7 +141,7 @@ export class MyChart extends PennLabsChart {
 		})
 
 		new CronJob(this, 'sync-path-course-statuses', {
-			schedule: cronTime.every(30).minutes(),
+			schedule: cronTime.every(5).minutes(),
 			image: backendImage,
 			secret,
 			cmd: ['python', 'manage.py', 'sync_path_status', '--slack'],