adds warning

Author: pcrespov

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

@@ -108,6 +108,7 @@ class ApplicationSettings(BaseCustomSettings, MixinLoggingSettings):
         env=["WEBSERVER_LOGLEVEL", "LOG_LEVEL", "LOGLEVEL"],
         # NOTE: suffix '_LOGLEVEL' is used overall
     )
+
     WEBSERVER_LOG_FORMAT_LOCAL_DEV_ENABLED: bool = Field(
         default=False,
         env=["WEBSERVER_LOG_FORMAT_LOCAL_DEV_ENABLED", "LOG_FORMAT_LOCAL_DEV_ENABLED"],

services/web/server/src/simcore_service_webserver/products/_middlewares.py

@@ -1,4 +1,5 @@
 import logging
+import textwrap
 from collections import OrderedDict
 
 from aiohttp import web
@@ -43,7 +44,7 @@ def _discover_product_by_request_header(request: web.Request) -> str | None:
     return None
 
 
-def _get_diagnose_msg(request: web.Request):
+def _get_debug_msg(request: web.Request):
     return "\n".join(
         [
             f"{request.url=}",
@@ -56,9 +57,6 @@ def _get_diagnose_msg(request: web.Request):
     )
 
 
-_INCLUDE_PATHS: set[str] = {"/static-frontend-data.json", "/socket.io/"}
-
-
 @web.middleware
 async def discover_product_middleware(request: web.Request, handler: Handler):
     """
@@ -71,8 +69,9 @@ async def discover_product_middleware(request: web.Request, handler: Handler):
     if (
         # - API entrypoints
         # - /static info for front-end
+        # - socket-io
         request.path.startswith(f"/{API_VTAG}")
-        or request.path in _INCLUDE_PATHS
+        or request.path in {"/static-frontend-data.json", "/socket.io/"}
     ):
         request[RQ_PRODUCT_KEY] = (
             _discover_product_by_request_header(request)
@@ -83,22 +82,20 @@ async def discover_product_middleware(request: web.Request, handler: Handler):
     else:
         # - Publications entrypoint: redirections from other websites. SEE studies_access.py::access_study
         # - Root entrypoint: to serve front-end apps
-        # request.path.startswith("/study/")
-        # or request.path.startswith("/view")
-        # or request.path == "/" )
+        assert (  # nosec
+            request.path.startswith("/dev/")
+            or request.path.startswith("/study/")
+            or request.path.startswith("/view")
+            or request.path == "/"
+        )
         request[RQ_PRODUCT_KEY] = _discover_product_by_hostname(
             request
         ) or _get_default_product_name(request.app)
 
-    # FIXME: session[product]
-    # if product_session := get_session(request).get('product_name'):
-    #  assert get_product_name(request) ==   product_session
-
+    # TODO: remove this!!!
     _logger.warning(
-        "\n%s\n%s\n%s\n",
-        "------------------TESTING-------------------",
-        _get_diagnose_msg(request),
-        "-------------------------------------------",
+        "Product middleware result: \n%s\n",
+        textwrap.indent(_get_debug_msg(request), " "),
     )
     assert request[RQ_PRODUCT_KEY]  # nosec