Running sentry with Ember embroider and fastboot breaks build

[oscarni]

• 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: @sentry/ember

Version:

5.30.0

Description

My ember app is running embroider 0.35.1 in fastboot. It works fine without Sentry or when fastboot is not enabled. But as soon as I add Sentry according to your latest official ember docs (adding InitSentryForEmber() in app.js) the build says:

Error: Assertion Failed: Missing configuration

Sentry is using ember-get-config. But it looks like the config is empty from where the sentry import is done.

sentry-javascript/packages/ember/addon/index.ts

Line 15 in 8f66ed8

const config = environmentConfig['@sentry/ember'];

Not sure if it's an embroider issue or if it's fixable here in the sentry repo. It looks like the config is not accessible from within fastboot when sentry is trying to access it. I also created an issue in the embroider repo embroider-build/embroider#642

[dashed]

@k-fish would you be able to respond to this?

[k-fish]

Hey there 👋 , I would expect that it might have something to do with the config being available in fastboot, embroider should be fine. Can you try with fastboot disabled?

[oscarni]

Hi, ok so sentry is not supported if I run fastboot?

As mentioned earlier it does work when run without fastboot. Eg visiting http://localhost:4200/?fastboot=false.

It also worked before updating to building with embroider.
I created a repo reproducing the issue https://github.com/oscarni/sentry-missing-config

[k-fish]

@oscarni sorry about that I missed the part about fastboot in your original post. The addon would work with Embroider and Fastboot separately, it just seemed to be an issue with ember-get-config creating a new file via broccoli likely causing a problem. I've removed ember-get-config and switched to using embroider macros, so it should work once #3181 is in. I'll close this once the new version is out.