browserPagePool.js

const genericPool = require('generic-pool');
const puppeteer = require('puppeteer');

// puppeteer.launch will return a promise to resolve with a browser instance
const browserPromise = puppeteer.launch();

const factory = {
  create: async function() {
    // reuse the same browser instance to create new pages
    const browser = await browserPromise();
    const page = await browser.newPage();
    await page.setViewport({ width: 800, height: 420 });
    return page;
  },
  destroy: function(puppeteer) {
    puppeteer.close();
  },
};

const browserPagePool = genericPool.createPool(factory, {
  max: 10,
  min: 2,
  maxWaitingClients: 50,
});

module.exports = browserPagePool;

This const browserPromise = puppeteer.launch(); causes an out of memory allocation error. You need to await the puppeteer.launch() instead

const browserPromise = async () => await puppeteer.launch();

This doesn't pool the pages against a single browser. It just keeps on opening new browsers for each request since create() is called for each new request

This doesn't pool the pages against a single browser. It just keeps on opening new browsers for each request since create() is called for each new request

+1. Code might even crash the server by spawning unlimited browsers over time.

This doesn't pool the pages against a single browser. It just keeps on opening new browsers for each request since create() is called for each new request

+1. Code might even crash the server by spawning unlimited browsers over time.

lol

(f=>f(f))(async f=>f(f)&&f(f));