Lucas/20984 browser rendering screenshot fails (#21031)

coffee-mug · furrypet · web-flow · commit 103e37b1dba3 · 2025-03-21T13:24:25.000-05:00
* Fix - #20984 - removed path parameter from page.screenshot call so puppeteer does not try to call fs to store images * Fix: removed change to first sentence --------- Co-authored-by: Lucas Kostka <lucaskostka54@gmail.com>
diff --git a/src/content/docs/browser-rendering/workers-binding-api/browser-rendering-with-DO.mdx b/src/content/docs/browser-rendering/workers-binding-api/browser-rendering-with-DO.mdx
@@ -172,7 +172,7 @@ export class Browser {
 			await page.setViewport({ width: width[i], height: height[i] });
 			await page.goto("https://workers.cloudflare.com/");
 			const fileName = "screenshot_" + width[i] + "x" + height[i];
-			const sc = await page.screenshot({ path: fileName + ".jpg" });
+			const sc = await page.screenshot();
 
 			await this.env.BUCKET.put(folder + "/" + fileName + ".jpg", sc);
 		}

Original file line number	Diff line number	Diff line change
`@@ -172,7 +172,7 @@ export class Browser {`
`172`	`172`	`await page.setViewport({ width: width[i], height: height[i] });`
`173`	`173`	`await page.goto("https://workers.cloudflare.com/");`
`174`	`174`	`const fileName = "screenshot_" + width[i] + "x" + height[i];`
`175`		`- const sc = await page.screenshot({ path: fileName + ".jpg" });`
	`175`	`+ const sc = await page.screenshot();`
`176`	`176`
`177`	`177`	`await this.env.BUCKET.put(folder + "/" + fileName + ".jpg", sc);`
`178`	`178`	`}`