Skip to content

Commit e6918a7

Browse files
committed
Minimal logging
1 parent 7002e0e commit e6918a7

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

scrapy_playwright/handler.py

+9
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import asyncio
2+
import logging
23
from time import time
34
from typing import Callable, Optional, Type, TypeVar
45
from urllib.parse import urlparse
@@ -25,6 +26,9 @@
2526
PlaywrightHandler = TypeVar("PlaywrightHandler", bound="ScrapyPlaywrightDownloadHandler")
2627

2728

29+
logger = logging.getLogger(__name__)
30+
31+
2832
def _make_request_handler(
2933
browser_type: str,
3034
scrapy_request: Request,
@@ -94,21 +98,26 @@ def from_crawler(cls: Type[PlaywrightHandler], crawler: Crawler) -> PlaywrightHa
9498
return cls(crawler)
9599

96100
def _engine_started(self) -> Deferred:
101+
logger.info("Launching browser")
97102
return deferred_from_coro(self._launch_browser())
98103

99104
async def _launch_browser(self) -> None:
100105
self.playwright_context_manager = playwright.AsyncPlaywrightContextManager()
101106
self.playwright = await self.playwright_context_manager.start()
102107
browser_launcher = getattr(self.playwright, self.browser_type).launch
103108
self.browser = await browser_launcher(**self.launch_options)
109+
logger.info(f"Browser {self.browser_type} launched with options: {self.launch_options}")
104110
self.context = await self.browser.newContext(**self.context_args)
111+
logger.info(f"Browser context started with args: {self.context_args}")
105112

106113
@inlineCallbacks
107114
def close(self) -> Deferred:
108115
yield super().close()
109116
if getattr(self, "context", None):
117+
logger.info("Closing browser context")
110118
yield deferred_from_coro(self.context.close())
111119
if getattr(self, "browser", None):
120+
logger.info("Closing browser")
112121
yield deferred_from_coro(self.browser.close())
113122
yield deferred_from_coro(self.playwright_context_manager.__aexit__())
114123

0 commit comments

Comments
 (0)