Puppeteer: يؤدي إغلاق المتصفح عند إغلاق الصفحة الأخيرة أحيانًا إلى حدوث خطأ

+1

بعد إجراء بعض التصحيح السريع للأخطاء ، تم تشغيل الحدث targetDestroyed قبل استلام رسالة page.close() (أو بالأحرى ، قام محرك الدمى بمعالجة الإقرار).

يمكنني التفكير في طرق لإصلاحها - على سبيل المثال ، انتظر الحصول على اعتراف Target.closeTarget قبل معالجة الحدث targetDestroyed . كيف يتم القيام بذلك بشكل صحيح رغم أنه سؤال آخر!

شكرا MJPA على التحقيق.

تأتي المشكلة من حقيقة أن browser يزيل الأهداف من القائمة قبل إغلاق الصفحات المرتبطة.

في المثال التالي ، أتوقع أن يقوم browser.pages() بالإبلاغ عن صفحة واحدة حتى تنتهي المكالمة 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 _b قبل_ إغلاق الصفحة ، وبالتالي فإن المثال التالي لا يعمل ، على غرار 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());  

أعتقد أن هذا _sort of_ منطقي ، لأن الحدث يسمى close وليس closed . لقد تمكنت من التغلب على هذا بمفردي عن طريق إصدار حدث closed في رد الاتصال بالوعد close() والاستماع إلى ذلك بدلاً من ذلك ، والذي _ يفعل_.

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

يبدو أن المغزى من القصة هو أنه لا توجد طريقة مضمونة 100٪ لمعرفة متى يكون المتصفح آمنًا للإغلاق (دون الحاجة إلى اكتشاف الاستثناء). يرجى إعلامي إذا فاتني شيء ما.

يحتمل أن تكون مرتبطة بـ # 3423؟

أحصل على هذا بعد إغلاق جميع علامات التبويب في المتصفح ثم محاولة فتح صفحة جديدة

في المثال التالي ، أتوقع أن يقوم browser.pages() بالإبلاغ عن صفحة واحدة حتى تنتهي المكالمة page.close() :

هذا من شأنه كسر الكود في المثال أعلاه. الصفحة الأخيرة مغلقة ، brower.pages (). سيكون الطول 1 ، ولن يتم إغلاق المتصفح. ثم لن تأتي المزيد من الأحداث المستهدفة المدمرة وسيظل المتصفح يعيش إلى الأبد.

أود أن أقول أن هذا خطأ مستخدم ، على الرغم من كونه دقيقًا. يجب انتظار أي مكالمات إلى 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();
}