summary refs log tree commit diff
diff options
context:
space:
mode:
authorDavid Baker <dave@matrix.org>2019-05-10 11:14:55 +0100
committerDavid Baker <dave@matrix.org>2019-05-10 11:14:55 +0100
commit9c61dce3c81283f9c7bb47b634512dd848eb116e (patch)
treee00ee53090827a19b9571a45c8b43ccad0e6e4d0
parentAdd changelog entry (diff)
downloadsynapse-9c61dce3c81283f9c7bb47b634512dd848eb116e.tar.xz
Comment
-rw-r--r--synapse/rest/client/v2_alpha/register.py4
1 files changed, 4 insertions, 0 deletions
diff --git a/synapse/rest/client/v2_alpha/register.py b/synapse/rest/client/v2_alpha/register.py
index 95578b76ae..5c4c8dca28 100644
--- a/synapse/rest/client/v2_alpha/register.py
+++ b/synapse/rest/client/v2_alpha/register.py
@@ -345,6 +345,10 @@ class RegisterRestServlet(RestServlet):
         if self.hs.config.enable_registration_captcha:
             # only support 3PIDless registration if no 3PIDs are required
             if not require_email and not require_msisdn:
+                # Also add a dummy flow here, otherwise if a client completes
+                # recaptcha first we'll assume they were going for this flow
+                # and complete the request, when they could have been trying to
+                # complete one of the flows with email/msisdn auth.
                 flows.extend([[LoginType.RECAPTCHA, LoginType.DUMMY]])
             # only support the email-only flow if we don't require MSISDN 3PIDs
             if not require_msisdn: