Sentry recording errors with beacon

[Jivings]

My Sentry is showing errors thrown in the SA script, looks like it's this line:

scripts/src/default.js

Line 484 in ed9efd1

nav[sendBeaconText](

Here's a link to the Sentry issue:

https://sentry.io/share/issue/20005bc054fb4e48a65b8c01209ae3d2/

[Jivings]

I think it's because you can't use the navigator object as a reference without binding it's scope. So you should be invoking it with call or apply.

nav[sendBeaconText].call(
  nav, 
  fullApiUrl + "/append",
  stringify(assign(payload, append))
);

So for example this doesn't work:

But this does:

[adriaandotcom]

Thanks for investigating this! Will update the script when I'm back from my holiday.

Maybe a better solution would be using sendBeacon directly:

nav.sendBeacon(
  fullApiUrl + "/append",
  stringify(assign(payload, append))
);

We don't need to size optimization if we need apply or call anyway.

[Jivings]

Yes that should be fine.

[adriaandotcom]

Fixed in #29