balena-io-experimental / meetbot

A Google Meet bot to make meetings frictionless
GNU Affero General Public License v3.0
32 stars 13 forks source link

Consider adding retries for page navigation #54

Open vipulgupta2048 opened 2 years ago

vipulgupta2048 commented 2 years ago

We hit navigation timeout quite a lot, and it is great to wrap these in a retry. I was thinking to use this https://www.npmjs.com/package/async-retry

28.12.21 19:56:37 (+0530)  main  ERROR in meet! Exiting... https://meet.google.com/oxd-hqer-qno TimeoutError: Navigation timeout of 30000 ms exceeded
28.12.21 19:56:37 (+0530)  main      at /usr/src/node_modules/puppeteer/lib/cjs/puppeteer/common/LifecycleWatcher.js:106:111
28.12.21 19:56:37 (+0530)  main      at async FrameManager.waitForFrameNavigation (/usr/src/node_modules/puppeteer/lib/cjs/puppeteer/common/FrameManager.js:128:23)
28.12.21 19:56:37 (+0530)  main      at async Frame.waitForNavigation (/usr/src/node_modules/puppeteer/lib/cjs/puppeteer/common/FrameManager.js:441:16)
28.12.21 19:56:37 (+0530)  main      at async Page.waitForNavigation (/usr/src/node_modules/puppeteer/lib/cjs/puppeteer/common/Page.js:1218:16)