Skip to content

Commit 794f618

Browse files
authored
Merge pull request #8 from psf/limit_to_registered_events
limit processing to events we actually listen for
2 parents ae9b3e1 + d5c21a3 commit 794f618

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

clabot/github.py

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,14 @@ class AsyncWebhookView(BaseAsyncWebhookView):
1818
async def post(self, request: HttpRequest) -> JsonResponse:
1919
event = self.get_event(request)
2020

21-
event_log = await EventLog.objects.acreate_from_event(event)
22-
23-
await self.router.adispatch(event, None)
24-
25-
return self.get_response(event_log)
21+
found_callbacks = self.router.fetch(event)
22+
if found_callbacks:
23+
event_log = await EventLog.objects.acreate_from_event(event)
24+
await self.router.adispatch(event, None)
25+
return self.get_response(event_log)
26+
else:
27+
return JsonResponse(
28+
{
29+
"message": "ok",
30+
}
31+
)

0 commit comments

Comments
 (0)