Logs: Redirect log in response (#589)

* redirect added

* resource type comparison added

* semicolon added

* Apply suggestions from code review

---------

Co-authored-by: Agnès Toulet <[email protected]>

src/browser/browser.ts

@@ -555,6 +555,7 @@ export class Browser {
       page.on('request', this.logRequest);
       page.on('requestfinished', this.logRequestFinished);
       page.on('close', this.logPageClosed);
+      page.on('response', this.logRedirectResponse);
     }
   }
 
@@ -568,6 +569,7 @@ export class Browser {
       page.off('request', this.logRequest);
       page.off('requestfinished', this.logRequestFinished);
       page.off('close', this.logPageClosed);
+      page.off('response', this.logRedirectResponse);
     }
   }
 
@@ -598,6 +600,14 @@ export class Browser {
     this.log.debug('Browser request', 'url', req.url(), 'method', req.method());
   };
 
+  logRedirectResponse = (resp: puppeteer.HTTPResponse) => {
+    const status = resp.status();
+    if (status >= 300 && status <= 399 && resp.request().resourceType() === 'document') {
+      const headers = resp.headers();
+      this.log.debug(`Redirect from ${resp.url()} to ${headers['location']}`);
+    }
+  };
+
   logRequestFailed = (req: any) => {
     let failureError = '';
     const failure = req?.failure();