@sanderegg review: aiohttp exception multiline

Author: pcrespov

packages/service-library/src/servicelib/aiohttp/rest_responses.py

@@ -92,7 +92,8 @@ def create_http_error(
     )
 
     return http_error_cls(
-        reason=reason,
+        # Multiline not allowed in HTTP reason
+        reason=reason.replace("\n", " ") if reason else None,
         text=json_dumps(
             payload,
         ),

services/web/server/src/simcore_service_webserver/exporter/utils.py

@@ -1,6 +1,7 @@
 import asyncio
+from collections.abc import Callable, Coroutine
 from pathlib import Path
-from typing import Any, Callable, Coroutine
+from typing import Any
 
 from aiohttp.abc import AbstractStreamWriter
 from aiohttp.typedefs import LooseHeaders
@@ -26,7 +27,8 @@ def __init__(
             path=path,
             chunk_size=chunk_size,
             status=status,
-            reason=reason,
+            # Multiline not allowed in HTTP reason
+            reason=reason.replace("\n", " ") if reason else None,
             headers=headers,
         )
         self.remove_tmp_dir_cb = remove_tmp_dir_cb