@GitHK review: msessage

Author: pcrespov

services/storage/tests/unit/test_utils_handlers.py

@@ -9,7 +9,6 @@
 
 import httpx
 import pytest
-from asyncpg import PostgresError
 from aws_library.s3._errors import S3AccessError, S3KeyNotFoundError
 from fastapi import FastAPI, HTTPException, status
 from fastapi.exceptions import RequestValidationError
@@ -28,6 +27,7 @@
     DatcoreAdapterTimeoutError,
 )
 from simcore_service_storage.modules.db.access_layer import InvalidFileIdentifierError
+from sqlalchemy.exc import DBAPIError
 
 
 @pytest.fixture
@@ -85,7 +85,12 @@ async def client(initialized_app: FastAPI) -> AsyncIterator[AsyncClient]:
             status.HTTP_422_UNPROCESSABLE_ENTITY,
         ),
         (
-            PostgresError("pytest postgres error"),
+            DBAPIError.instance(
+                statement="pytest statement",
+                params={},
+                orig=Exception("pytest original"),
+                dbapi_base_err=Exception("pytest dbapi base error"),
+            ),
             status.HTTP_503_SERVICE_UNAVAILABLE,
         ),
         (

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

@@ -50,7 +50,7 @@ def _handle_exceptions_as_503():
     except DatabaseError as err:
         _logger.exception(
             **create_troubleshooting_log_kwargs(
-                MSG_AUTH_NOT_AVAILABLE + ": Auth unavailable due to database error.",
+                f"{MSG_AUTH_NOT_AVAILABLE}: Auth unavailable due to database error.",
                 error=err,
                 error_context={"origin": str(err.orig) if err.orig else None},
                 tip="Check database connection",