Puppeteer: 最後のページを閉じたときにブラウザを閉じると、エラーがスローされることがあります

+1

少し簡単にデバッグしましたが、 page.close()メッセージが確認される前に（つまり、パペッティアが確認を処理する前に） targetDestroyedイベントが発生します。

私はそれを修正する方法を考えることができます-例えば、 targetDestroyedイベントを処理する前にTarget.closeTarget確認応答を待ちます。 それを適切に行う方法は別の質問です！

調査してくれた@MJPAに感謝します。

この問題は、関連するページが閉じられる前にbrowserがリストからターゲットを削除するという事実に起因します。

次の例では、 page.close()呼び出しが終了するまで、 browser.pages()が1ページをレポートすることを期待しています。

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

この場合、常に同じエラーが発生しました。

多分このパペッティアエラー。

少し似た実装でも同じエラーが発生しますが、 page closeイベントをリッスンします。 ドキュメントは「閉じる」の意味について過度に明示的ではありませんが、その実装によれば、 closeはページが閉じられる前に起動されるため、 targetdestroyedように次の例は機能しません。

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

イベントはclosedではなくcloseと呼ばれるので、この_sortof_は理にかなっていると思います。 close() promiseコールバックで自分のclosedイベントを発行し、代わりにそれをリッスンすることで、これを回避しました。これは機能します。

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

話の教訓は、ブラウザがいつ安全に閉じることができるかを知る100％保証された方法がないようです（例外をキャッチする必要はありません）。 足りないものがあれば教えてください。

おそらく＃3423に関連していますか？

ブラウザのすべてのタブを閉じてから新しいページを開こうとすると、これが表示されます

次の例では、 page.close()呼び出しが終了するまで、 browser.pages()が1ページをレポートすることを期待しています。

これにより、上記の例のコードが破損します。 最後に閉じられたページ、brower.pages（）。lengthは1になり、ブラウザーは閉じられません。 そうすれば、ターゲットが破壊されたイベントが発生しなくなり、ブラウザは永久に存続します。

微妙ですが、これはユーザーエラーだと思います。 page.close呼び出しは、 browser.close()呼び出す前に待機する必要があります。

ユーザーが制御するページがない場合は、Promise.allのすべてのページを閉じてから、ブラウザーを閉じます。 targetdestroyedイベントを聞く必要はありません。 または、browser.closeを呼び出して、すべてのページを自動的に閉じます。

ユーザーが制御するページがある場合は、 targetdestroyedイベントをリッスンしますが、 page.closeを呼び出す必要はありません。

+1。
page.on("popup")ポップアップを閉じた後、このエラーが表示され始めました。 フレームコンテキスト内でポップアップが発生すると、「popup.close（）」でポップアップを閉じることができず、パペッティアがタブを開いたままにすることがあるため、ブラウザ全体を閉じようとすると、「TargetClosed」エラーがスローされます。スクリプトが停止します。

Chrome73でpuppeteer1.14を使用しています（Chromiumは使用していません）
ブラウザを閉じるために使用するコード

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