registration

Author: pcrespov

services/web/server/src/simcore_service_webserver/login/handlers_registration.py

@@ -9,6 +9,7 @@
 from pydantic import BaseModel, Field, PositiveInt, SecretStr, validator
 from servicelib.aiohttp import status
 from servicelib.aiohttp.requests_validation import parse_request_body_as
+from servicelib.logging_errors import create_troubleshotting_log_kwargs
 from servicelib.mimetype_constants import MIMETYPE_APPLICATION_JSON
 from simcore_postgres_database.models.users import UserStatus
 
@@ -58,7 +59,7 @@
 )
 from .utils_email import get_template_path, send_email_from_template
 
-log = logging.getLogger(__name__)
+_logger = logging.getLogger(__name__)
 
 
 routes = RouteTableDef()
@@ -266,7 +267,7 @@ async def register(request: web.Request):
             )
         except Exception as err:  # pylint: disable=broad-except
             error_code = create_error_code(err)
-            log.exception(
+            _logger.exception(
                 "Failed while sending confirmation email to %s, %s [%s]",
                 f"{user=}",
                 f"{_confirmation=}",
@@ -400,13 +401,18 @@ async def register_phone(request: web.Request):
     except Exception as err:  # pylint: disable=broad-except
         # Unhandled errors -> 503
         error_code = create_error_code(err)
-        log.exception(
-            "Phone registration failed [%s]",
-            f"{error_code}",
-            extra={"error_code": error_code},
+        user_msg = f"Currently we cannot register phone numbers [{error_code}]"
+
+        _logger.exception(
+            **create_troubleshotting_log_kwargs(
+                user_msg,
+                exception=err,
+                error_context={"request": request, "registration": registration},
+                tip="Phone registration failed",
+            )
         )
 
         raise web.HTTPServiceUnavailable(
-            reason=f"Currently we cannot register phone numbers ({error_code})",
+            reason=user_msg,
             content_type=MIMETYPE_APPLICATION_JSON,
         ) from err