From 65133c9d9d06423602d823cd98114482d55bd4eb Mon Sep 17 00:00:00 2001 From: Ilya Kreymer Date: Mon, 4 Mar 2024 23:58:39 -0800 Subject: [PATCH] resourceType lowercase fix: (#483) follow up to #481, check reqresp.resourceType with lowercase value just set message based on resourceType value --- src/util/recorder.ts | 21 +++++++-------------- 1 file changed, 7 insertions(+), 14 deletions(-) diff --git a/src/util/recorder.ts b/src/util/recorder.ts index 6765b0777..242ecbd91 100644 --- a/src/util/recorder.ts +++ b/src/util/recorder.ts @@ -635,20 +635,13 @@ export class Recorder { body, }); } catch (e) { - const type = reqresp.resourceType; - if (type === "Document") { - logger.debug( - "document not loaded in browser, possibly other URLs missing", - { url, type: reqresp.resourceType }, - "recorder", - ); - } else { - logger.debug( - "URL not loaded in browser", - { url, type: reqresp.resourceType }, - "recorder", - ); - } + const { resourceType } = reqresp; + const msg = + resourceType === "document" + ? "document not loaded in browser, possibly other URLs missing" + : "URL not loaded in browser"; + + logger.debug(msg, { url, resourceType }, "recorder"); } return true;