RegEx in tracingOrigins config breaks ember build

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which package are you using?

@sentry/ember

SDK Version

7.14.1

Framework Version

ember 4.7.0

Link to Sentry event

No response

Steps to Reproduce

1. Create new app using ember-cli 4.7.0 ember new ember-quickstart --lang en
2. install sentry SDK ember install @sentry/ember
3. add config to config/environment.js, make sure to include RegEx in tracingOrigins, e.g.

  ENV['@sentry/ember'] = {
    sentry: {
      tracesSampleRate: 1.0,
      debug: false,
      browserTracingOptions: {
        tracingOrigins: [
          'localhost',
          /^\//,
        ],
      },
    },
  };

1. run ember ember serve

Expected Result

ember builds and runs app

Actual Result

build fails with error

non serializable item found in config: /^\//
[Embroider:MacrosConfig] the given config from '/YOUR_PATH/node_modules/@sentry/ember' for packageName 'undefined' is not JSON serializable.

[mydea]

I’d say both are possible.

We can also leverage (=copy) the validation that also happens in embroider (which, incidentally, I wrote there for exactly this issue 😂 see https://github.com/embroider-build/embroider/pull/1083). but, as you can see there, this is not entirely straightforward, and would also kind of duplicate all of this functionality/checks. But might be a bit more “user friendly” this way, as theoretically you may also want to set other config based on env (e.g. debug, or similar).

Anyhow, for me both approaches seem OK! The more user friendly would be to validate it and throw a nicer error message, I guess.

[k-fish]

It should be fine using the regex as is recommended in the docs, using Sentry.init and calling it within the application. Using Ember config to pass Sentry options can’t serialize functions / objects etc. which is why the regex is breaking. Try switching to pass the Sentry options via Sentry.init?

@AbhiPrasad, we should probably have a deprecated warning for passing the sentry options via config, considering it’s come up before.