2FA: Fix check for context_data attr.

This attribute only makes sense for responses that are generated using a
template.

zerver/tests/test_signup.py

@@ -503,6 +503,10 @@ class LoginTest(ZulipTestCase):
         response = self.client_get("/login/")
         self.assertEqual(response["Location"], "http://zulip.testserver")
 
+    def test_options_request_to_login_page(self) -> None:
+        response = self.client_options('/login/')
+        self.assertEqual(response.status_code, 200)
+
 class InviteUserBase(ZulipTestCase):
     def check_sent_emails(self, correct_recipients: List[str],
                           custom_from_name: Optional[str]=None) -> None:

zerver/views/auth.py

@@ -11,6 +11,7 @@ from zerver.decorator import authenticated_json_post_view, require_post, \
     process_client, do_login, log_view_func
 from django.http import HttpRequest, HttpResponse, HttpResponseRedirect, \
     HttpResponseNotFound
+from django.template.response import SimpleTemplateResponse
 from django.middleware.csrf import get_token
 from django.shortcuts import redirect, render
 from django.views.decorators.csrf import csrf_exempt
@@ -578,12 +579,14 @@ def login_page(request: HttpRequest, **kwargs: Any) -> HttpResponse:
         assert len(e.args) > 1
         return redirect_to_misconfigured_ldap_notice(e.args[1])
 
-    if isinstance(template_response, HttpResponseRedirect):
-        # We return immediately; redirect responses don't have a
-        # `.context_data` to update with update_login_page_context.
-        return template_response
+    if isinstance(template_response, SimpleTemplateResponse):
+        # Only those responses that are rendered using a template have
+        # context_data attribute. This attribute doesn't exist otherwise. It is
+        # added in SimpleTemplateResponse class, which is a derived class of
+        # HttpResponse. See django.template.response.SimpleTemplateResponse,
+        # https://github.com/django/django/blob/master/django/template/response.py#L19.
+        update_login_page_context(request, template_response.context_data)
 
-    update_login_page_context(request, template_response.context_data)
     return template_response
 
 @csrf_exempt