Merge pull request #98 from pwa-builder/dev

Disabling speed optimizations on Report
pwa-builder · Dec 13, 2023 · f4e077f · f4e077f
2 parents b8aad89 + a34a16e
commit f4e077f
Show file tree

Hide file tree

Showing 3 changed files with 42 additions and 41 deletions.
diff --git a/Report/index.ts b/Report/index.ts
@@ -17,7 +17,7 @@ import { ChildProcess, exec, spawn } from 'child_process';
 const __dirname = dirname(fileURLToPath(import.meta.url));
 // const _root = `${__dirname}/../..`;
 
-const AZURE_FUNC_TIMEOUT = 2 * 60 * 1000;
+const AZURE_FUNC_TIMEOUT = 3 * 60 * 1000;
 const SPAWN_TIMEOUT = AZURE_FUNC_TIMEOUT - 15 * 1000;
 
 const httpTrigger: AzureFunction = async function (

diff --git a/Report/lighthouse/lighthouse.ts b/Report/lighthouse/lighthouse.ts
@@ -137,46 +137,48 @@ async function execute() {
   });
 
   page.on('request', (req) => {
-      const resourceType = req.resourceType();
-      if (SKIP_RESOURCES.some((type) => resourceType == type)){
-          switch (resourceType) {
-            case 'image':
-              req.respond(
-                {
-                  status: 200,
-                  contentType: 'image/jpeg',
-                  body: Buffer.from(base64Image, 'base64'),
-                });
-              break;
-            case 'font':
-              req.respond(
-                {
-                  status: 200,
-                  contentType: 'font/woff2',
-                  body: Buffer.from(base64Font, 'base64'),
+      // commented because it doesn't work on Azure environment
+      // const resourceType = req.resourceType();
+      // if (SKIP_RESOURCES.some((type) => resourceType == type)){
+      //     let contentType = req.headers().accept?.split(',')[0] || 'text/plain';
+      //     switch (resourceType) {
+      //       case 'image':
+      //         req.respond(
+      //           {
+      //             status: 200,
+      //             contentType,
+      //             body: Buffer.from(base64Image, 'base64'),
+      //           });
+      //         break;
+      //       case 'font':
+      //         req.respond(
+      //           {
+      //             status: 200,
+      //             contentType,
+      //             body: Buffer.from(base64Font, 'base64'),
 
-                });
-              break;
-            case 'fetch':
-              if (req.method() == 'GET')
-                req.respond({
-                  status: 200,
-                  contentType: 'application/json',
-                  body: JSON.stringify({ success: true, message: "Intercepted fetch request" }),
-                });
-              else
-                req.continue();
-              break;
-            default:
-              req.respond({
-                status: 200,
-                contentType: 'text/plain',
-                body: '{"success": true}',
-              });
-              break;
-          }
-      }
-      else
+      //           });
+      //         break;
+      //       case 'fetch':
+      //         if (req.method() == 'GET')
+      //           req.respond({
+      //             status: 200,
+      //             contentType,
+      //             body: JSON.stringify({ success: true, message: "Intercepted fetch request" }),
+      //           });
+      //         else
+      //           req.continue();
+      //         break;
+      //       default:
+      //         req.respond({
+      //           status: 200,
+      //           contentType,
+      //           body: '{"success": true}',
+      //         });
+      //         break;
+      //     }
+      // }
+      // else
         req.continue(); 
   });
 

diff --git a/tests/test_urls.json b/tests/test_urls.json
@@ -8,7 +8,6 @@
 "https://nute.app/",
 "https://garfieldapp.tk/",
 "https://microsoftedge.github.io/Demos/wami/",
-"https://app.mapbuddy.app/",
 "https://breedershub.in",
 "https://ril.kryptografische.biz/",
 "https://cryptojam.net",