Puppeteer: Закрытие браузера при закрытии последней страницы иногда вызывает ошибку

+1

Выполнив небольшую быструю отладку, событие targetDestroyed запускается до того, как сообщение page.close() было подтверждено (или, скорее, кукловод обработал подтверждение).

Я могу придумать способы исправить это - например, дождаться подтверждения Target.closeTarget перед обработкой события targetDestroyed . Как это сделать - другой вопрос!

Спасибо @MJPA за расследование.

Проблема возникает из-за того, что browser удаляет цели из списка до закрытия связанных страниц.

В следующем примере я ожидаю, что browser.pages() сообщит об 1 странице, пока вызов 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();
})();

Выход:

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

@JoelEinbinder, что ты думаешь?

Та же проблема, что и @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()
    ]);

В этом случае всегда возникали одни и те же ошибки.

Может это ошибка кукловода.

Я получаю ту же ошибку с немного похожей реализацией, но при прослушивании события close в page . В документации не слишком четко указано, что означает «закрыть», но в соответствии с его реализацией close запускается _перед_ закрытием страницы, поэтому следующий пример не работает, аналогично targetdestroyed example.

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());  

Я предполагаю, что это _sort of_ имеет смысл, поскольку событие называется close а не closed . Я обошел это самостоятельно, выполнив собственное событие closed в обратном вызове close() обещания и послушав это вместо этого, что _действует_.

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

Мораль этой истории, похоже, заключается в том, что не существует 100% гарантированного способа узнать, когда браузер безопасно закрыть (без необходимости перехватывать исключение). Пожалуйста, дайте мне знать, если я что-то упускаю.

Возможно связано с № 3423?

Я получаю это после закрытия всех вкладок в браузере, а затем попытки открыть новую страницу

В следующем примере я ожидаю, что browser.pages() сообщит об 1 странице, пока вызов page.close() будет завершен:

Это нарушит код в приведенном выше примере. Последняя закрытая страница, brower.pages (). Length будет равна 1, браузер не будет закрыт. Тогда больше не будет событий об уничтожении цели, и браузер будет жить вечно.

Я бы сказал, что это ошибка пользователя, хотя и незаметная. Перед вызовом browser.close() следует дождаться любых вызовов page.close browser.close() .

Если страниц, управляемых пользователем, нет, просто Promise.all закроет все страницы, а затем закройте браузер. Не должно быть необходимости прослушивать события targetdestroyed . Или просто вызовите browser.close и позвольте всем страницам закрыться самостоятельно.

Если есть страницы, управляемые пользователем, то слушайте события targetdestroyed , но вызывать page.close не нужно.

+1.
Я начал видеть эту ошибку после закрытия всплывающих окон с помощью page.on("popup") . Когда всплывающее окно срабатывает внутри контекста фрейма, иногда я не могу закрыть его с помощью «popup.close ()», и кукольник оставляет вкладку открытой, поэтому, когда я пытаюсь закрыть весь браузер, он выдает ошибку «Цель закрыта», и сценарий остановится.

Я использую кукольник 1.14 с Chrome 73 (я не использую Chromium)
Код, который я использую для закрытия браузера

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