Skip to content

Commit ac0071a

Browse files
committed
test: simplify status retrieval in request logging for clarity
1 parent 76b9852 commit ac0071a

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

pydoll/browser/requests/request.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,8 +141,9 @@ async def request(
141141
received_headers = self._extract_received_headers()
142142
sent_headers = self._extract_sent_headers()
143143
cookies = self._extract_set_cookies()
144+
status = result.get('result', {}).get('result', {}).get('value', {}).get('status')
144145
logger.debug(
145-
f'Request complete: status={result["result"]["result"]["value"]["status"]}, '
146+
f'Request complete: status={status}, '
146147
f'received_headers={len(received_headers)}, sent_headers={len(sent_headers)}, '
147148
f'cookies={len(cookies)}'
148149
)

pydoll/connection/managers/events_manager.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,10 +84,10 @@ async def process_event(self, event_data: CDPEvent):
8484

8585
if 'Network.requestWillBeSent' in event_name:
8686
self._update_network_logs(event_data)
87-
logger.debug(
88-
f'Network log appended (total={len(self.network_logs)}): '
89-
f'{event_data["params"].get("request", {}).get("url", "")}'
87+
req_url = event_data.get('params', {}).get('request', {}).get('url') or event_data.get(
88+
'url', ''
9089
)
90+
logger.debug(f'Network log appended (total={len(self.network_logs)}): {req_url}')
9191

9292
if 'Page.javascriptDialogOpening' in event_name:
9393
self.dialog = JavascriptDialogOpeningEvent(

0 commit comments

Comments
 (0)