Use assert_param_in_dict

1 files changed, 2 insertions, 6 deletions

diff --git a/synapse/handlers/auth.py b/synapse/handlers/auth.py

index 57dead89b5..fd4e4148e3 100644
--- a/synapse/handlers/auth.py
+++ b/synapse/handlers/auth.py
@@ -39,6 +39,7 @@ from synapse.api.ratelimiting import Ratelimiter
 from synapse.handlers.ui_auth import INTERACTIVE_AUTH_CHECKERS
 from synapse.handlers.ui_auth.checkers import UserInteractiveAuthChecker
 from synapse.http.server import finish_request
+from synapse.http.servlet import assert_params_in_dict
 from synapse.http.site import SynapseRequest
 from synapse.logging.context import defer_to_thread
 from synapse.metrics.background_process_metrics import run_as_background_process
@@ -78,12 +79,7 @@ def client_dict_convert_legacy_fields_to_identifier(
 
     # We've converted valid, legacy login submissions to an identifier. If the
     # dict still doesn't have an identifier, it's invalid
-    if "identifier" not in submission:
-        raise SynapseError(
-            400,
-            "Missing 'identifier' parameter in login submission",
-            errcode=Codes.MISSING_PARAM,
-        )
+    assert_params_in_dict(submission, required=["identifier"])
 
     # Ensure the identifier has a type
     if "type" not in submission["identifier"]: