WIP Upgrade dependencies for security issues

ampproject · May 29, 2024 · 53a9041 · 53a9041
1 parent c7d6eae
commit 53a9041
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 13 deletions.
diff --git a/packages/optimizer/spec/end-to-end/EndToEndSpec.js b/packages/optimizer/spec/end-to-end/EndToEndSpec.js
@@ -29,6 +29,20 @@ const fetch = fetchMock
   .mock('https://example.com/amp/rtv/123456789000000/v0.css', '/* example.com v0.css */')
   .mock('https://cdn.ampproject.org/v0.css', '/* ampproject.org v0.css */');
 
+const nock = require('nock');
+nock('https://cdn.ampproject.org').get('/v0/validator.json').reply(200, '{}');
+const CACHES_JSON = `{
+  "caches": [
+    {
+      "id": "google",
+      "name": "Google AMP Cache",
+      "docs": "https://developers.google.com/amp/cache/",
+      "updateCacheApiDomainSuffix": "cdn.ampproject.org"
+    }
+  ]
+}`;
+nock('https://cdn.ampproject.org').get('/caches.json').reply(200, CACHES_JSON);
+
 createSpec({
   name: 'End-to-End: AMP First',
   testDir: __dirname,

diff --git a/packages/optimizer/spec/helpers/validatorInstance.js b/packages/optimizer/spec/helpers/validatorInstance.js
@@ -6,7 +6,6 @@ let instance = null;
 module.exports = {
   get: () => {
     if (!instance) {
-      console.error('Validator instance created: ' + path.join(__dirname, 'validator.js'));
       instance = validator.getInstance(path.join(__dirname, 'validator.js'));
     }
     return instance;

diff --git a/packages/page-experience/lib/PageDataGatherer.js b/packages/page-experience/lib/PageDataGatherer.js
@@ -58,20 +58,16 @@ class PageAnalyzer {
       throw new Error('Puppeteer not running, please call `start` first.');
     }
     const {page, remoteStyles, responsePromise} = await this.setupPage();
-    try {
-      await page.goto(url, {waitUntil: 'load'});
+    await page.goto(url, {waitUntil: 'load'});
 
-      const response = await responsePromise;
-      if (!response) {
-        throw new Error('Failed loading url', url);
-      }
-      const {html, headers} = response;
-      return await this.gatherPageData(page, {remoteStyles, html, headers});
-    } finally {
-      if (page) {
-        page.close();
-      }
+    const response = await responsePromise;
+    if (!response) {
+      throw new Error('Failed loading url', url);
     }
+    const {html, headers} = response;
+    const data = await this.gatherPageData(page, {remoteStyles, html, headers});
+    await page.close();
+    return data;
   }
 
   /**
@@ -80,6 +76,7 @@ class PageAnalyzer {
   async shutdown() {
     try {
       await this.browser.close();
+      this.browser = null;
     } catch (e) {
       console.log(e);
     }