Use whitespace in default redirect logger name instead of -

Author: Pijukatel

src/apify_client/_logging.py

@@ -164,5 +164,5 @@ def format(self, record: logging.LogRecord) -> str:
         Returns:
             Formated log message.
         """
-        formated_logger_name = f'{Fore.CYAN}[{record.name}]{Style.RESET_ALL} '
-        return f'{formated_logger_name}-> {record.msg}'
+        formated_logger_name = f'{Fore.CYAN}[{record.name}]{Style.RESET_ALL}'
+        return f'{formated_logger_name} -> {record.msg}'

src/apify_client/clients/resource_clients/run.py

@@ -281,7 +281,7 @@ def get_streamed_log(self, to_logger: logging.Logger | None = None, *, from_star
         actor_name = actor_data.get('name', '') if run_data else ''
 
         if not to_logger:
-            name = '-'.join(part for part in (actor_name, run_id) if part)
+            name = ' '.join(part for part in (actor_name, run_id) if part)
             to_logger = create_redirect_logger(f'apify.{name}')
 
         return StreamedLogSync(log_client=self.log(), to_logger=to_logger, from_start=from_start)
@@ -345,7 +345,7 @@ def get_status_message_watcher(
         actor_name = actor_data.get('name', '') if run_data else ''
 
         if not to_logger:
-            name = '-'.join(part for part in (actor_name, run_id) if part)
+            name = ' '.join(part for part in (actor_name, run_id) if part)
             to_logger = create_redirect_logger(f'apify.{name}')
 
         return StatusMessageWatcherSync(run_client=self, to_logger=to_logger, check_period=check_period)
@@ -605,7 +605,7 @@ async def get_streamed_log(
         actor_name = actor_data.get('name', '') if run_data else ''
 
         if not to_logger:
-            name = '-'.join(part for part in (actor_name, run_id) if part)
+            name = ' '.join(part for part in (actor_name, run_id) if part)
             to_logger = create_redirect_logger(f'apify.{name}')
 
         return StreamedLogAsync(log_client=self.log(), to_logger=to_logger, from_start=from_start)
@@ -670,7 +670,7 @@ async def get_status_message_watcher(
         actor_name = actor_data.get('name', '') if run_data else ''
 
         if not to_logger:
-            name = '-'.join(part for part in (actor_name, run_id) if part)
+            name = ' '.join(part for part in (actor_name, run_id) if part)
             to_logger = create_redirect_logger(f'apify.{name}')
 
         return StatusMessageWatcherAsync(run_client=self, to_logger=to_logger, check_period=check_period)

tests/unit/test_logging.py

@@ -34,7 +34,7 @@
     b'2025-05-13T07:27:14.132Z [apify] DEB',  # Chunked log that got split outside of marker
     b'UG e\n',  # part 2
     # Already redirected message
-    b'2025-05-13T07:28:14.132Z [apify.redirect-logger-4U1oAnKau6jpzjUuA] -> 2025-05-13T07:27:14.132Z ACTOR: Pulling\n',
+    b'2025-05-13T07:28:14.132Z [apify.redirect-logger runId:4U1oAnKau6jpzjUuA] -> 2025-05-13T07:27:14.132Z ACTOR:...\n',
 )
 _EXISTING_LOGS_BEFORE_REDIRECT_ATTACH = 3
 
@@ -49,7 +49,8 @@
     ('2025-05-13T07:26:14.132Z [apify] DEBUG d', logging.DEBUG),
     ('2025-05-13T07:27:14.132Z [apify] DEBUG e', logging.DEBUG),
     (
-        '2025-05-13T07:28:14.132Z [apify.redirect-logger-4U1oAnKau6jpzjUuA] -> 2025-05-13T07:27:14.132Z ACTOR: Pulling',
+        '2025-05-13T07:28:14.132Z [apify.redirect-logger runId:4U1oAnKau6jpzjUuA] -> '
+        '2025-05-13T07:27:14.132Z ACTOR:...',
         logging.INFO,
     ),
 )
@@ -265,7 +266,7 @@ async def test_actor_call_redirect_logs_to_default_logger_async(
     """Test that logs are redirected correctly to the default logger.
 
     Caplog contains logs before formatting, so formatting is not included in the test expectations."""
-    logger_name = f'apify.{_MOCKED_ACTOR_NAME}-runId:{_MOCKED_RUN_ID}'
+    logger_name = f'apify.{_MOCKED_ACTOR_NAME} runId:{_MOCKED_RUN_ID}'
     logger = logging.getLogger(logger_name)
     actor_client = ApifyClientAsync(token='mocked_token', api_url=_MOCKED_API_URL).actor(actor_id=_MOCKED_ACTOR_ID)
 
@@ -293,7 +294,7 @@ def test_actor_call_redirect_logs_to_default_logger_sync(
     """Test that logs are redirected correctly to the default logger.
 
     Caplog contains logs before formatting, so formatting is not included in the test expectations."""
-    logger_name = f'apify.{_MOCKED_ACTOR_NAME}-runId:{_MOCKED_RUN_ID}'
+    logger_name = f'apify.{_MOCKED_ACTOR_NAME} runId:{_MOCKED_RUN_ID}'
     logger = logging.getLogger(logger_name)
     actor_client = ApifyClient(token='mocked_token', api_url=_MOCKED_API_URL).actor(actor_id=_MOCKED_ACTOR_ID)