From 36623cddfd41e3ff7a19e83f0235300a2dfd83f8 Mon Sep 17 00:00:00 2001 From: Puneeth Chaganti Date: Fri, 17 Apr 2020 17:20:18 +0530 Subject: [PATCH] webhooks/dialogflow: Remove default value for email parameter. The webhook view used a default value for the email, which gave non-informative errors when the webhook is incorrectly configured without the email parameter. --- zerver/webhooks/dialogflow/view.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/zerver/webhooks/dialogflow/view.py b/zerver/webhooks/dialogflow/view.py index a22fd18148..d87ba13745 100644 --- a/zerver/webhooks/dialogflow/view.py +++ b/zerver/webhooks/dialogflow/view.py @@ -14,7 +14,7 @@ from zerver.models import UserProfile, get_user_profile_by_email @has_request_variables def api_dialogflow_webhook(request: HttpRequest, user_profile: UserProfile, payload: Dict[str, Any]=REQ(argument_type='body'), - email: str=REQ(default='foo')) -> HttpResponse: + email: str=REQ()) -> HttpResponse: status = payload["status"]["code"] if status == 200: