Puppeteer: Cerrar el navegador cuando se cierra la última página a veces arroja un error

+1

Hizo un poco de depuración rápida, el evento targetDestroyed se activa antes de que se reconozca el mensaje page.close() (o más bien, el titiritero haya procesado el reconocimiento).

Puedo pensar en formas de solucionarlo, por ejemplo, esperar un reconocimiento de Target.closeTarget antes de manejar el evento targetDestroyed . ¡Sin embargo, cómo hacerlo correctamente es otra cuestión!

Gracias @MJPA por la investigación.

El problema proviene del hecho de que browser elimina los objetivos de la lista antes de que se cierren las páginas asociadas.

En el siguiente ejemplo, esperaría que browser.pages() informe 1 página hasta que finalice la llamada page.close() :

const puppeteer = require('puppeteer');

(async () => {
  const browser = await puppeteer.launch();
  const [page] = await browser.pages();
  browser.on('targetdestroyed', async () => console.log('Target destroyed. Pages count: ' + (await browser.pages()).length));
  console.log('closing page...');
  await page.close();
  console.log('page closed.');
  await browser.close();
})();

La salida:

closing page...
Target destroyed. Pages count: 0
page closed.

@JoelEinbinder ¿qué opinas?

Mismo problema que @MJPA

        await page.waitForSelector('aside.aside a[href="/checkout"]')
    .then(() => {
        logMessage('Checkout is available');
    })
    .catch(err => {
        log('Checkout not available');
        success = false;
        browser.close();
    });
    // click checkout button
    await Promise.all([
        page.click('aside.aside a[href="/checkout"]'),
        page.waitForNavigation()
    ]);

En este caso, siempre ocurrieron los mismos errores.

Quizás este error del titiritero.

Recibo el mismo error con una implementación ligeramente similar, pero al escuchar el evento close en un page . La documentación no es demasiado explícita sobre lo que significa "cerrar", pero de acuerdo con su implementación, close se activa _antes_ de que se cierre la página, por lo que el siguiente ejemplo no funciona, similar al targetdestroyed ejemplo.

const puppeteer = require('puppeteer');
const browser = await puppeteer.launch();

// create 5 test pages
for(let i=0; i<5; i++){

  let page = await browser.newPage();

  // listen to page close event and close browser
  // when all get closed
  page.on('close', async () => {
    let pages = await browser.pages();
    if(!pages.length){
      // will not get here!
      browser.close();
    }
  });

}

let pages = await browser.pages();

pages.forEach(async (page) => await page.close());  

Supongo que este _sort of_ tiene sentido, ya que el evento se llama close y no closed . Solucioné esto por mi cuenta emitiendo mi propio evento closed en la devolución de llamada de promesa close() y escuchando eso en su lugar, que _ sí_ funciona.

page.close().then(() => {
  page.emit('closed');
});

La moraleja de la historia parece ser que no existe un método 100% garantizado para saber cuándo es seguro cerrar un navegador (sin tener que detectar la excepción). Avísame si me falta algo.

¿Posiblemente relacionado con el # 3423?

Recibo esto después de cerrar todas las pestañas en un navegador y luego intentar abrir una nueva página

En el siguiente ejemplo, esperaría que browser.pages() informe 1 página hasta que finalice la llamada page.close() :

Esto rompería el código del ejemplo anterior. La última página cerrada, brower.pages (). Length sería 1, el navegador no se cerraría. Entonces no vendrían más eventos de destrucción de objetivos y el navegador viviría para siempre.

Yo diría que esto es un error del usuario, aunque es sutil. Cualquier llamada a page.close debe esperarse antes de llamar a browser.close() .

Si no hay páginas controladas por el usuario, simplemente Promise.all se cierra toda la página y luego cierra el navegador. No debería ser necesario escuchar los eventos targetdestroyed . O simplemente llame a browser.close y deje que todas las páginas se cierren solas.

Si hay páginas controladas por el usuario, escuche los eventos targetdestroyed , pero no debería ser necesario llamar a page.close .

+1.
Comencé a ver este error después de cerrar las ventanas emergentes con page.on("popup") . Cuando una ventana emergente se dispara dentro de un contexto de Frame, a veces no puedo cerrarla con "popup.close ()" y el titiritero deja la pestaña abierta, así que cuando intento cerrar todo el navegador, arroja un error de "Objetivo cerrado", y el guión se detiene.

Yo uso titiritero 1.14, con Chrome 73 (no uso Chromium)
El código que utilizo para cerrar el navegador.

async close_browser() {
    for (let page of await this.browser.pages()) {
      await page.close({
        "runBeforeUnload": true
      });
    }
   await this.browser.close();
}