Merge pull request #13 from elgentos/WouterSteen-patch-2

Remove Headless Browsers from pushing data to RUMVision

Author: WouterSteen

src/view/frontend/templates/script/rumvision.phtml

@@ -15,16 +15,35 @@ if (! $viewModel->shouldIncludeScript()) {
 ?>
 
 <script>
-    window.rumv = window.rumv || function() { (window.rumv.q = window.rumv.q || []).push(arguments) };
-    (function(rum, vi,si,on) {
-        var s = JSON.parse( sessionStorage.getItem('rumv') || '{"pageviews":0}' ); s.pageviews++;
-        if ( s.urls && s.regex && ( s.page = eval('('+s.regex+')')( s.urls, vi.location.pathname ) ) && !s.page.type ) {
-            return sessionStorage.setItem('rumv', JSON.stringify( s ) );
-        }
-
-        vi.rumv.storage = s;
-        var head = si.querySelector('head'), js = si.createElement('script');
-        js.src = 'https://d5yoctgpv4cpx.cloudfront.net/'+rum+'/v4-'+vi.location.hostname+'.js';
-        head.appendChild(js);
-    })( '<?= $escaper->escapeJs($viewModel->getTrackingId()) ?>', window, document, '<?= $escaper->escapeJs($viewModel->getHostName()) ?>' );
+    (function() {
+        const isHeadless = navigator.userAgent.includes('HeadlessChrome')
+        || navigator.webdriver
+        || !window.chrome
+        || /Headless/.test(navigator.userAgent);
+
+        if (isHeadless) return;
+
+        window.rumv = window.rumv || function() {
+            (window.rumv.q = window.rumv.q || []).push(arguments);
+        };
+
+        (function(rum, vi, si, on) {
+            var s = JSON.parse(sessionStorage.getItem('rumv') || '{"pageviews":0}');
+            s.pageviews++;
+
+            if (s.urls && s.regex && (s.page = eval('(' + s.regex + ')')(s.urls, vi.location.pathname)) && !s.page.type) {
+                return sessionStorage.setItem('rumv', JSON.stringify(s));
+            }
+
+            vi.rumv.storage = s;
+            var head = si.querySelector('head'), js = si.createElement('script');
+            js.src = 'https://d5yoctgpv4cpx.cloudfront.net/' + rum + '/v4-' + vi.location.hostname + '.js';
+            head.appendChild(js);
+        })(
+            '<?= $escaper->escapeJs($viewModel->getTrackingId()) ?>',
+            window,
+            document,
+            '<?= $escaper->escapeJs($viewModel->getHostName()) ?>'
+        );
+    })();
 </script>