fix: preserve query and fragment in WebSocket URL for tabs

Author: thalissonvs

pydoll/browser/chromium/base.py

@@ -821,14 +821,21 @@ def _get_tab_kwargs(self, target_id: str, browser_context_id: Optional[str] = No
 
     def _get_tab_ws_address(self, tab_id: str) -> str:
         """
-        Get WebSocket address for tab. If tab_id is not provided,
-        it will be derived from the targets.
+        Get WebSocket address for a specific tab, preserving any query or fragment
+        components present in the original browser-level WebSocket URL.
+
+        This ensures authentication tokens passed via query string (e.g.,
+        ws://host/devtools/browser/abc?token=XYZ) are retained when switching
+        to the page-level endpoint (devtools/page/<tab_id>), which is critical
+        for providers like Browserless or authenticated CDP proxies.
         """
         if not self._ws_address:
             raise InvalidWebSocketAddress('WebSocket address is not set')
 
-        ws_domain = '/'.join(self._ws_address.split('/')[:3])
-        return f'{ws_domain}/devtools/page/{tab_id}'
+        parts = urlsplit(self._ws_address)
+        # Preserve scheme and netloc; build the page path and keep query/fragment
+        page_path = f'/devtools/page/{tab_id}'
+        return urlunsplit((parts.scheme, parts.netloc, page_path, parts.query, parts.fragment))
 
     @staticmethod
     def _sanitize_proxy_and_extract_auth(

pydoll/browser/managers/browser_process_manager.py

@@ -55,7 +55,7 @@ def start_browser_process(
     @staticmethod
     def _default_process_creator(command: list[str]) -> subprocess.Popen:
         """Create browser process with output capture to prevent console clutter."""
-        return subprocess.Popen(command, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
+        return subprocess.Popen(command, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
 
     def stop_process(self):
         """