fixes messages errors

Author: pcrespov

packages/service-library/src/servicelib/logging_errors.py

@@ -39,10 +39,10 @@ def _collect_causes(exc: BaseException) -> str:
     debug_data = json_dumps(
         {
             "exception_type": f"{type(error)}",
-            "exception_details": f"{error}",
+            "exception_string": f"{error}",
+            "exception_causes": _collect_causes(error),
             "error_code": error_code,
             "context": error_context,
-            "causes": _collect_causes(error),
             "tip": tip,
         },
         default=representation_encoder,

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

@@ -150,9 +150,9 @@ def _get_error_context(
             # do not want to forward but rather log due to the special rules in this entrypoint
             _logger.warning(
                 **create_troubleshotting_log_kwargs(
-                    f"{_error_msg_prefix} for invalid user. {_error_msg_suffix}.",
+                    f"{_error_msg_prefix} for invalid user. {err.text}. {_error_msg_suffix}",
                     error=err,
-                    error_context=_get_error_context(user),
+                    error_context={**_get_error_context(user), "error.text": err.text},
                 )
             )
             ok = False
@@ -168,7 +168,7 @@ def _get_error_context(
         ):
             _logger.warning(
                 **create_troubleshotting_log_kwargs(
-                    f"{_error_msg_prefix} for a user with NO access to this product. {_error_msg_suffix}.",
+                    f"{_error_msg_prefix} for a user with NO access to this product. {_error_msg_suffix}",
                     error=Exception("User cannot access this product"),
                     error_context=_get_error_context(user),
                 )

services/web/server/src/simcore_service_webserver/security/_authz_web.py

@@ -6,7 +6,7 @@
 from models_library.users import UserID
 
 from ._authz_access_model import AuthContextDict, OptionalContext
-from ._constants import PERMISSION_PRODUCT_LOGIN_KEY
+from ._constants import MSG_UNAUTHORIZED, PERMISSION_PRODUCT_LOGIN_KEY
 
 assert PERMISSION_PRODUCT_LOGIN_KEY  # nosec
 
@@ -28,7 +28,7 @@ async def check_user_authorized(request: web.Request) -> UserID:
     # NOTE: Same as aiohttp_security.api.check_authorized
     user_id: UserID | None = await aiohttp_security.api.authorized_userid(request)
     if user_id is None:
-        raise web.HTTPUnauthorized
+        raise web.HTTPUnauthorized(text=MSG_UNAUTHORIZED)
     return user_id
 
 

services/web/server/src/simcore_service_webserver/security/_constants.py

@@ -1,5 +1,5 @@
 from typing import Final
 
 MSG_AUTH_NOT_AVAILABLE: Final[str] = "Authentication service is temporary unavailable"
-
+MSG_UNAUTHORIZED: Final[str] = "Unauthorized"
 PERMISSION_PRODUCT_LOGIN_KEY: Final[str] = "product.login"