Merge pull request #23 from Gavant/upgrade-node-20

upgrade to node 20 and other misc improvements

.github/workflows/ci.yml

@@ -20,7 +20,7 @@ jobs:
             - uses: actions/checkout@v3
             - uses: volta-cli/action@v1
               with:
-                  node-version: 18.x
+                  node-version: 20.x
                   yarn-version: 1.x
             - name: Install Dependencies
               run: yarn install

package.json

@@ -50,7 +50,7 @@
         "access": "public"
     },
     "volta": {
-        "node": "18.16.0",
+        "node": "20.9.0",
         "yarn": "3.6.4"
     }
 }

src/services/pdf-generation.ts

@@ -1,3 +1,4 @@
+import { executablePath } from 'puppeteer';
 import puppeteer, { PDFOptions } from 'puppeteer-core';
 import report from 'puppeteer-report';
 
@@ -41,11 +42,12 @@ export default class PdfGenerationService {
     }
 
     async launchBrowser(pdfGenerationRequest: PdfGenerationRequest) {
-        const chromiumPath = await chromium.executablePath();
+        const chromiumPath = process.env.IS_LOCAL ? executablePath() : await chromium.executablePath();
         const options = {
-            args: chromium.args,
+            args: process.env.IS_LOCAL ? process.env.BROWSER_ARGS?.split(',') ?? ['--no-sandbox'] : chromium.args,
+            defaultViewport: chromium.defaultViewport,
             executablePath: chromiumPath,
-            headless: !!process?.env?.BROWSER_HEADLESS === false ? false : true,
+            headless: chromium.headless as boolean,
             ...pdfGenerationRequest.browserOptions
         };