fix(scrapeURL/fire-engine): wait longer if timeout is not specified
This commit is contained in:
@@ -96,7 +96,9 @@ export async function scrapeURLWithFireEngineChromeCDP(meta: Meta): Promise<Engi
|
|||||||
let response = await performFireEngineScrape(
|
let response = await performFireEngineScrape(
|
||||||
meta.logger.child({ method: "scrapeURLWithFireEngineChromeCDP/callFireEngine", request }),
|
meta.logger.child({ method: "scrapeURLWithFireEngineChromeCDP/callFireEngine", request }),
|
||||||
request,
|
request,
|
||||||
defaultTimeout + totalWait,
|
meta.options.timeout !== undefined
|
||||||
|
? defaultTimeout + totalWait
|
||||||
|
: Infinity, // TODO: better timeout handling
|
||||||
);
|
);
|
||||||
|
|
||||||
specialtyScrapeCheck(meta.logger.child({ method: "scrapeURLWithFireEngineChromeCDP/specialtyScrapeCheck" }), response.responseHeaders);
|
specialtyScrapeCheck(meta.logger.child({ method: "scrapeURLWithFireEngineChromeCDP/specialtyScrapeCheck" }), response.responseHeaders);
|
||||||
@@ -148,7 +150,9 @@ export async function scrapeURLWithFireEnginePlaywright(meta: Meta): Promise<Eng
|
|||||||
let response = await performFireEngineScrape(
|
let response = await performFireEngineScrape(
|
||||||
meta.logger.child({ method: "scrapeURLWithFireEngineChromeCDP/callFireEngine", request }),
|
meta.logger.child({ method: "scrapeURLWithFireEngineChromeCDP/callFireEngine", request }),
|
||||||
request,
|
request,
|
||||||
defaultTimeout + meta.options.waitFor
|
meta.options.timeout !== undefined
|
||||||
|
? defaultTimeout + meta.options.waitFor
|
||||||
|
: Infinity, // TODO: better timeout handling
|
||||||
);
|
);
|
||||||
|
|
||||||
specialtyScrapeCheck(meta.logger.child({ method: "scrapeURLWithFireEnginePlaywright/specialtyScrapeCheck" }), response.responseHeaders);
|
specialtyScrapeCheck(meta.logger.child({ method: "scrapeURLWithFireEnginePlaywright/specialtyScrapeCheck" }), response.responseHeaders);
|
||||||
@@ -189,6 +193,9 @@ export async function scrapeURLWithFireEngineTLSClient(meta: Meta): Promise<Engi
|
|||||||
let response = await performFireEngineScrape(
|
let response = await performFireEngineScrape(
|
||||||
meta.logger.child({ method: "scrapeURLWithFireEngineChromeCDP/callFireEngine", request }),
|
meta.logger.child({ method: "scrapeURLWithFireEngineChromeCDP/callFireEngine", request }),
|
||||||
request,
|
request,
|
||||||
|
meta.options.timeout !== undefined
|
||||||
|
? defaultTimeout
|
||||||
|
: Infinity, // TODO: better timeout handling
|
||||||
);
|
);
|
||||||
|
|
||||||
specialtyScrapeCheck(meta.logger.child({ method: "scrapeURLWithFireEngineTLSClient/specialtyScrapeCheck" }), response.responseHeaders);
|
specialtyScrapeCheck(meta.logger.child({ method: "scrapeURLWithFireEngineTLSClient/specialtyScrapeCheck" }), response.responseHeaders);
|
||||||
|
|||||||
Reference in New Issue
Block a user