Should the puppeteer timeout value be something that is user configurable?

[brandon-dacrib]

I see lots of puppeteer timouts:
Creating Meter
Calling meter.last_read()..
Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/coned/meter.py", line 96, in last_read
await self.browse()
File "/usr/local/lib/python3.7/dist-packages/coned/meter.py", line 172, in browse
await page.waitForNavigation()
File "/usr/local/lib/python3.7/dist-packages/pyppeteer/page.py", line 906, in waitForNavigation
raise error
pyppeteer.errors.TimeoutError: Navigation Timeout Exceeded: 30000 ms exceeded.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "run.py", line 26, in
startTime, endTime, value, uom = asyncio.get_event_loop().run_until_complete(meter.last_read())
File "/usr/lib/python3.7/asyncio/base_events.py", line 584, in run_until_complete
return future.result()
File "/usr/local/lib/python3.7/dist-packages/coned/meter.py", line 118, in last_read
raise MeterError("Error requesting meter data")
coned.meter.MeterError: Error requesting meter data
Sleeping 5 mins..

And I suspect this is because the con-ed site is genuinely slow from my experience browsing manually. Can/should you make that timeout something that is user-configurable?

https://pocketadmin.tech/en/puppeteer-timeout/