Revert "Send an email if the address is already bound to an user account (element-hq#16819)"

This reverts commit ae18123.

Author: 3np

synapse/config/emailconfig.py

@@ -52,7 +52,6 @@
     "invite_from_person_to_space": "[%(app)s] %(person)s has invited you to join the %(space)s space on %(app)s...",
     "password_reset": "[%(server_name)s] Password reset",
     "email_validation": "[%(server_name)s] Validate your email",
-    "email_already_in_use": "[%(server_name)s] Email already in use",
 }
 
 LEGACY_TEMPLATE_DIR_WARNING = """
@@ -77,7 +76,6 @@ class EmailSubjectConfig:
     invite_from_person_to_space: str
     password_reset: str
     email_validation: str
-    email_already_in_use: str
 
 
 class EmailConfig(Config):
@@ -183,12 +181,6 @@ def read_config(self, config: JsonDict, **kwargs: Any) -> None:
             registration_template_text = email_config.get(
                 "registration_template_text", "registration.txt"
             )
-            already_in_use_template_html = email_config.get(
-                "already_in_use_template_html", "already_in_use.html"
-            )
-            already_in_use_template_text = email_config.get(
-                "already_in_use_template_html", "already_in_use.txt"
-            )
             add_threepid_template_html = email_config.get(
                 "add_threepid_template_html", "add_threepid.html"
             )
@@ -224,8 +216,6 @@ def read_config(self, config: JsonDict, **kwargs: Any) -> None:
                 self.email_password_reset_template_text,
                 self.email_registration_template_html,
                 self.email_registration_template_text,
-                self.email_already_in_use_template_html,
-                self.email_already_in_use_template_text,
                 self.email_add_threepid_template_html,
                 self.email_add_threepid_template_text,
                 self.email_password_reset_template_confirmation_html,
@@ -241,8 +231,6 @@ def read_config(self, config: JsonDict, **kwargs: Any) -> None:
                     password_reset_template_text,
                     registration_template_html,
                     registration_template_text,
-                    already_in_use_template_html,
-                    already_in_use_template_text,
                     add_threepid_template_html,
                     add_threepid_template_text,
                     "password_reset_confirmation.html",

synapse/push/mailer.py

@@ -205,22 +205,6 @@ async def send_registration_mail(
             template_vars,
         )
 
-    emails_sent_counter.labels("already_in_use")
-
-    async def send_already_in_use_mail(self, email_address: str) -> None:
-        """Send an email if the address is already bound to an user account
-
-        Args:
-            email_address: Email address we're sending to the "already in use" mail
-        """
-
-        await self.send_email(
-            email_address,
-            self.email_subjects.email_already_in_use
-            % {"server_name": self.hs.config.server.server_name, "app": self.app_name},
-            {},
-        )
-
     emails_sent_counter.labels("add_threepid")
 
     async def send_add_threepid_mail(

synapse/res/templates/already_in_use.html

@@ -86,18 +86,12 @@ def __init__(self, hs: "HomeServer"):
         self.config = hs.config
 
         if self.hs.config.email.can_verify_email:
-            self.registration_mailer = Mailer(
+            self.mailer = Mailer(
                 hs=self.hs,
                 app_name=self.config.email.email_app_name,
                 template_html=self.config.email.email_registration_template_html,
                 template_text=self.config.email.email_registration_template_text,
             )
-            self.already_in_use_mailer = Mailer(
-                hs=self.hs,
-                app_name=self.config.email.email_app_name,
-                template_html=self.config.email.email_already_in_use_template_html,
-                template_text=self.config.email.email_already_in_use_template_text,
-            )
 
     async def on_POST(self, request: SynapseRequest) -> Tuple[int, JsonDict]:
         if not self.hs.config.email.can_verify_email:
@@ -145,10 +139,8 @@ async def on_POST(self, request: SynapseRequest) -> Tuple[int, JsonDict]:
             if self.hs.config.server.request_token_inhibit_3pid_errors:
                 # Make the client think the operation succeeded. See the rationale in the
                 # comments for request_token_inhibit_3pid_errors.
-                # Still send an email to warn the user that an account already exists.
                 # Also wait for some random amount of time between 100ms and 1s to make it
                 # look like we did something.
-                await self.already_in_use_mailer.send_already_in_use_mail(email)
                 await self.hs.get_clock().sleep(random.randint(1, 10) / 10)
                 return 200, {"sid": random_string(16)}
 
@@ -159,7 +151,7 @@ async def on_POST(self, request: SynapseRequest) -> Tuple[int, JsonDict]:
             email,
             client_secret,
             send_attempt,
-            self.registration_mailer.send_registration_mail,
+            self.mailer.send_registration_mail,
             next_link,
         )
 

synapse/res/templates/already_in_use.txt

@@ -22,7 +22,6 @@
 import datetime
 import os
 from typing import Any, Dict, List, Tuple
-from unittest.mock import AsyncMock
 
 import pkg_resources
 
@@ -43,7 +42,6 @@
 from synapse.util import Clock
 
 from tests import unittest
-from tests.server import ThreadedMemoryReactorClock
 from tests.unittest import override_config
 
 
@@ -60,13 +58,6 @@ def default_config(self) -> Dict[str, Any]:
         config["allow_guest_access"] = True
         return config
 
-    def make_homeserver(
-        self, reactor: ThreadedMemoryReactorClock, clock: Clock
-    ) -> HomeServer:
-        hs = super().make_homeserver(reactor, clock)
-        hs.get_send_email_handler()._sendmail = AsyncMock()
-        return hs
-
     def test_POST_appservice_registration_valid(self) -> None:
         user_id = "@as_user_kermit:test"
         as_token = "i_am_an_app_service"