ref: Remove keepalive:true as a default (#1496)

kamilogorek · web-flow · commit 0aed3ac9a773 · 2018-09-04T12:44:25.000+02:00
diff --git a/packages/raven-js/src/raven.js b/packages/raven-js/src/raven.js
@@ -98,7 +98,6 @@ function Raven() {
   };
   this._fetchDefaults = {
     method: 'POST',
-    keepalive: true,
     // Despite all stars in the sky saying that Edge supports old draft syntax, aka 'never', 'always', 'origin' and 'default
     // https://caniuse.com/#feat=referrer-policy
     // It doesn't. And it throw exception instead of ignoring this parameter...
diff --git a/packages/raven-js/test/raven.test.js b/packages/raven-js/test/raven.test.js
@@ -1870,7 +1870,6 @@ describe('globals', function() {
           assert.deepEqual(window.fetch.lastCall.args, [
             'http://localhost/?a=1&b=2',
             {
-              keepalive: true,
               referrerPolicy: supportsReferrerPolicy() ? 'origin' : '',
               method: 'POST',
               body: '{"foo":"bar"}'

Original file line number	Diff line number	Diff line change
`@@ -1870,7 +1870,6 @@ describe('globals', function() {`
`1870`	`1870`	`assert.deepEqual(window.fetch.lastCall.args, [`
`1871`	`1871`	`'http://localhost/?a=1&b=2',`
`1872`	`1872`	`{`
`1873`		`- keepalive: true,`
`1874`	`1873`	`referrerPolicy: supportsReferrerPolicy() ? 'origin' : '',`
`1875`	`1874`	`method: 'POST',`
`1876`	`1875`	`body: '{"foo":"bar"}'`