Skip to content

Commit b105b68

Browse files
committed
wait for network finish instead of page.waitFor, should fix emtpy tiles in screnshots, updated browser args
1 parent d0775b9 commit b105b68

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

vue/screenshot-service/screenshot-server.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,10 @@ const browserFactory = {
1818
devtools: false,
1919
args: [
2020
'--no-sandbox',
21-
'--disable-dev-shm-usage'
21+
'--disable-dev-shm-usage',
22+
'--disable-setuid-sandbox',
23+
'--disable-accelerated-2d-canvas',
24+
'--disable-gpu'
2225
// '--remote-debugging-port=9222',
2326
// '--remote-debugging-address=0.0.0.0'
2427
]
@@ -71,9 +74,12 @@ app.get('*', async (req, res) => {
7174

7275
const page = await browserPool.acquire();
7376
page.setViewport(pageOptions);
74-
await page.goto(screenshotUrl);
77+
await page.goto(screenshotUrl, {
78+
waitLoad: true,
79+
waitNetworkIdle: true
80+
});
7581
await page.waitForSelector('#mapinfo .v-list-item__title');
76-
await page.waitFor(1000); // extra 1sec wait, for map tiles
82+
// await page.waitFor(1000); // extra 1sec wait, for map tiles
7783
const screenshotBuffer = await page.screenshot({ encoding: 'binary' });
7884
// await browser.close();
7985
await browserPool.release(page);

0 commit comments

Comments
 (0)