Allow for specifying pluggable integrations with Lazy Loader directly from CDN

[dryoma]

I know that in order to use pluggable integrations with the CDN version a corresponding script needs to be included on a page:

<script src="https://browser.sentry-cdn.com/5.0.2/dedupe.min.js" crossorigin="anonymous"></script>

Is it also the recommended way to use non-default integrations with the Lazy Loader? Won't the discrepancy between the automatically set version in the Loader and the fixed version in integrations scripts be an issue? Maybe there is a way to specify the integrations I want to use as parameters to the Loaders scr as, say, Google Fonts do? Or maybe something like https://browser.sentry-cdn.com/5.x/dedupe.min.js would be possible so that the version of an integration could correspond to the verison that the Loader fetches?

[kamilogorek]

Is it also the recommended way to use non-default integrations with the Lazy Loader?

It's fine to use it like this, although try to use the latest version, eg. https://browser.sentry-cdn.com/5.13.2/dedupe.min.js for dedupe.

Won't the discrepancy between the automatically set version in the Loader and the fixed version in integrations scripts be an issue?

Not if you'll stick with the same major version, which can be selected in the project settings. Eg 5.x instead of the latest, but we don't plan to release v6 anytime soon anyway. Right now, the only thing you have to remember is to configure the integration in the provided callback:

Sentry.onLoad(function() {
  Sentry.init({
    integrations: [new Sentry.Integrations.Dedupe()]
  });
});

Maybe there is a way to specify the integrations I want to use as parameters to the Loaders scr as, say, Google Fonts do?

Unfortunately not, as we didn't have such a feature request yet. It'd also take quite a lot of work to make it work correctly.

Also, one thing worth noting is that loader is also open-sourced and you can create your own bundle of Sentry+Dedupe and point the loader to load it instead of CDN version - https://github.com/getsentry/sentry-javascript/blob/master/packages/browser/src/loader.js

Hope that helps! :)
Cheers!

[dryoma]

Thanks @kamilogorek ,

you can create your own bundle of Sentry+Dedupe

We thought about it, but decided that we'd rather use the official build. Especially since the version discrepancy won't be an issue.

Unfortunately not, as we didn't have such a feature request yet.

Can this ticket be considered as one? ;)

[kamilogorek]

Done :)

[dryoma]

Thanks @kamilogorek

Just stumbled upon another reason why we'd like the ability to bundle non-default Integrations on demand. When loading corresponding scripts for them, the scripts sometimes would fail to load. The same happens for our scripts as well sometimes, my guess is that some network issues are the culprit. But the result is that errors like this keep popping up. The nasty thing about them is that we can't even group them because Sentry fails to initialize our config because of that error, so notifications keep coming. There are options, like checking if the integration has loaded and only then initialize it; but I really don't want to end up with a client that spams thousands of errors without Dedupe keeping it at bay.

[github-actions]

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀