⚡ Ignore modulepreload in link tags

DataDog · Jan 5, 2023 · a3774ef · a3774ef
1 parent ac26b8d
commit a3774ef
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/packages/rum/src/domain/record/privacy.spec.ts b/packages/rum/src/domain/record/privacy.spec.ts
@@ -203,6 +203,16 @@ describe('getNodeSelfPrivacyLevel', () => {
       html: '<script class="dd-privacy-allow">',
       expected: NodePrivacyLevel.ALLOW,
     },
+    {
+      msg: 'is a link with rel=preload and as=script',
+      html: '<link rel="preload" crossorigins as="script">',
+      expected: NodePrivacyLevel.IGNORE,
+    },
+    {
+      msg: 'is a link with rel=modulepreload and as=script',
+      html: '<link rel="modulepreload" as="script">',
+      expected: NodePrivacyLevel.IGNORE,
+    },
 
     // Precedence
     {

diff --git a/packages/rum/src/domain/record/privacy.ts b/packages/rum/src/domain/record/privacy.ts
@@ -220,8 +220,8 @@ export function shouldIgnoreElement(element: Element): boolean {
   if (element.nodeName === 'LINK') {
     const relAttribute = getLowerCaseAttribute('rel')
     return (
-      // Scripts
-      (relAttribute === 'preload' && getLowerCaseAttribute('as') === 'script') ||
+      // Link as script - Ignore only when rel=preload or modulepreload
+      (/preload/i.test(relAttribute) && getLowerCaseAttribute('as') === 'script') ||
       // Favicons
       relAttribute === 'shortcut icon' ||
       relAttribute === 'icon'