Improved logging around browser initialization and retries

Author: cvasseng

lib/browser.js

@@ -78,6 +78,7 @@ export async function create(puppeteerArgs) {
 
   // Create a browser
   if (!browser) {
+    const maxTries = 25;
     let tryCount = 0;
 
     const open = async () => {
@@ -88,18 +89,29 @@ export async function create(puppeteerArgs) {
         );
         browser = await puppeteer.launch(launchOptions);
       } catch (error) {
+        // This isn't a full error yet as puppeteer sometimes takes time to
+        // initialize properly.
         logWithStack(
-          1,
+          2,
           error,
-          '[browser] Failed to launch a browser instance.'
+          `[browser] Failed to launch a browser instance - retrying (attempt ${tryCount}/${maxTries}).`
         );
 
         // Retry to launch browser until reaching max attempts
         if (tryCount < 25) {
-          log(3, `[browser] Retry to open a browser (${tryCount} out of 25).`);
+          log(
+            3,
+            `[browser] Retry to open a browser (attempt ${tryCount}/${maxTries}).`
+          );
           await new Promise((response) => setTimeout(response, 4000));
           await open();
         } else {
+          //... now it's an error
+          logWithStack(
+            1,
+            error,
+            '[browser] Failed to load browser after maximum retries reached.'
+          );
           throw error;
         }
       }