From 82ce88e4481e69f564b41ccbc6a11b5b42fa4bf6 Mon Sep 17 00:00:00 2001
From: mkomorski <marcin.komorski@pollub.edu.pl>
Date: Tue, 4 Jun 2024 18:59:14 +0200
Subject: [PATCH] Default to fetch keepalive (#11682)

Co-authored-by: Marcin Komorski <marcinkomorski@Marcins-MacBook-Pro.local>
---
 modules/genericAnalyticsAdapter.js                | 2 +-
 src/ajax.js                                       | 3 +++
 test/spec/modules/genericAnalyticsAdapter_spec.js | 4 ++--
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/modules/genericAnalyticsAdapter.js b/modules/genericAnalyticsAdapter.js
index 7f721863912..ce37e5c02fe 100644
--- a/modules/genericAnalyticsAdapter.js
+++ b/modules/genericAnalyticsAdapter.js
@@ -142,7 +142,7 @@ export function defaultHandler({url, method, batchSize, ajax = ajaxBuilder()}) {
   const serialize = method === 'GET' ? (data) => ({data: JSON.stringify(data)}) : (data) => JSON.stringify(data);
 
   return function (events) {
-    ajax(url, callbacks, serialize(extract(events)), {method})
+    ajax(url, callbacks, serialize(extract(events)), {method, keepalive: true})
   }
 }
 
diff --git a/src/ajax.js b/src/ajax.js
index ef4c2e4bcb4..92bff6dd527 100644
--- a/src/ajax.js
+++ b/src/ajax.js
@@ -52,6 +52,9 @@ export function toFetchRequest(url, data, options = {}) {
     // but we're not in a secure context
     rqOpts.browsingTopics = true;
   }
+  if (options.keepalive) {
+    rqOpts.keepalive = true;
+  }
   return dep.makeRequest(url, rqOpts);
 }
 
diff --git a/test/spec/modules/genericAnalyticsAdapter_spec.js b/test/spec/modules/genericAnalyticsAdapter_spec.js
index 2d9c7b4ae45..f574a33bf86 100644
--- a/test/spec/modules/genericAnalyticsAdapter_spec.js
+++ b/test/spec/modules/genericAnalyticsAdapter_spec.js
@@ -265,7 +265,7 @@ describe('Generic analytics', () => {
           handler([payload, {}]);
           sinon.assert.calledWith(ajax, url, sinon.match.any,
             sinon.match(data => sinon.match(payload).test(parse(data))),
-            {method}
+            {method, keepalive: true}
           );
         });
 
@@ -275,7 +275,7 @@ describe('Generic analytics', () => {
           handler(payload);
           sinon.assert.calledWith(ajax, url, sinon.match.any,
             sinon.match(data => sinon.match(payload).test(parse(data))),
-            {method}
+            {method, keepalive: true}
           );
         });
       });