fix: enhance error logging in change email process and correct variable assignment in API base URL

Author: pcrespov

services/web/server/src/simcore_service_webserver/login/_controller/rest/change.py

@@ -9,6 +9,7 @@
 
 from ...._meta import API_VTAG
 from ....db.plugin import get_asyncpg_engine
+from ....exception_handling import create_error_context_from_request
 from ....products import products_web
 from ....products.models import Product
 from ....users import users_service
@@ -258,7 +259,19 @@ async def initiate_change_email(request: web.Request):
             },
         )
     except Exception as err:  # pylint: disable=broad-except
-        _logger.exception("Can not send change_email_email")
+        _logger.exception(
+            **create_troubleshooting_log_kwargs(
+                "Can not send change_email_email",
+                error=err,
+                error_context={
+                    "user_id": user["id"],
+                    "user_email": user["email"],
+                    "new_email": request_body.email,
+                    "product_name": product.name,
+                    **create_error_context_from_request(request),
+                },
+            )
+        )
         await confirmation_service.delete_confirmation(confirmation)
         raise web.HTTPServiceUnavailable(text=MSG_CANT_SEND_MAIL) from err
 

services/web/server/src/simcore_service_webserver/utils_aiohttp.py

@@ -171,5 +171,5 @@ def iter_origins(request: web.Request) -> Iterator[tuple[str, str]]:
 
 def get_api_base_url(request: web.Request) -> str:
     scheme, host = next(iter_origins(request))
-    api_host = api_host = host if is_ip_address(host) else f"api.{host}"
+    api_host = host if is_ip_address(host) else f"api.{host}"
     return f"{scheme}://{api_host}"