FR: use multiple browser client AND utilize useragent integartion

[daisy1754]

• Review the documentation: https://docs.sentry.io/
• Search for existing issues: https://github.com/getsentry/sentry-javascript/issues
• Use the latest release: https://github.com/getsentry/sentry-javascript/releases
• Provide a link to the affected event from your Sentry account

Package + Version

• @sentry/browser
• @sentry/node
• raven-js
• raven-node (raven for node)
• other:

Version:

5.21.4

Description

We want to embed sentry in the JS library. Our client may already using sentry on their site so our library initialize its own browser client like below:

client = new Sentry.BrowserClient({
        dsn: sentryReportingURL,
        release: version,
        environment: process.env.NODE_ENV,
        integrations: Sentry.defaultIntegrations,
    });

// later
client.captureException(e);

We noticed userAgent is missing in sentry and found closed issue #1764 and https://github.com/getsentry/sentry-javascript/blob/master/packages/browser/src/integrations/useragent.ts#L24

Looks like userAgent integration depends on so we cannot send userAgent without using currenthub. We don't want to override currentHub because we are library and not override any of client's state.

Can UserAgent integration and other default integration not depend on currentHub?

[kamilogorek]

Hi @daisy1754 can you give this code a try?

const client = new Sentry.BrowserClient({
  dsn: sentryReportingURL,
  release: version,
  environment: process.env.NODE_ENV,
  integrations: Sentry.defaultIntegrations,
});
const hub = new Sentry.Hub(client);

hub.run(() => hub.captureException(e));

[daisy1754]

@kamilogorek it worked, thanks for help!

[daisy1754]

just following up I had to revert above because of #2622