hardcode crossorigin script attribute

fusionjs · Apr 23, 2019 · bdbc84f · bdbc84f
1 parent ae778bd
commit bdbc84f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 16 deletions.
diff --git a/plugins/ssr-plugin.js b/plugins/ssr-plugin.js
@@ -114,8 +114,8 @@ const SSRBodyTemplate = createPlugin/*:: <SSRBodyTemplateDepsType,SSRBodyTemplat
 
         for (let url of criticalChunkUrls) {
           const crossoriginAttr = url.startsWith(__webpack_public_path__)
-            ? ''
-            : ' crossorigin="anonymous"';
+            ? ' crossorigin="anonymous"'
+            : '';
           preloadHints.push(
             `<link rel="preload" href="${url}" nonce="${
               ctx.nonce

diff --git a/test/e2e/dynamic-import-app/test.js b/test/e2e/dynamic-import-app/test.js
@@ -175,13 +175,6 @@ test('`fusion build` app with dynamic imports integration', async () => {
     'one extra script after loading new route'
   );
 
-  t.ok(
-    await page.$$eval('script[src]:not([type="application/json"])', els =>
-      els.every(el => el.crossOrigin === null)
-    ),
-    'non-module scripts do not have crossorigin attribute'
-  );
-
   t.ok(
     await page.$$eval('script[src]:not([type="application/json"])', els =>
       // eslint-disable-next-line
@@ -274,13 +267,6 @@ test('`fusion build` app with Safari user agent and same-origin', async () => {
     'one extra script after loading new route'
   );
 
-  t.ok(
-    await page.$$eval('script[src]:not([type="application/json"])', els =>
-      els.every(el => el.crossOrigin === null)
-    ),
-    'non-module scripts do not have crossorigin attribute'
-  );
-
   t.ok(
     await page.$$eval('script[src]:not([type="application/json"])', els =>
       // eslint-disable-next-line