Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Put in default Sentry DSN #2240

Closed
ianb opened this issue Feb 28, 2017 · 1 comment
Closed

Put in default Sentry DSN #2240

ianb opened this issue Feb 28, 2017 · 1 comment
Assignees
@ianb
Milestone
General Release 55

Comments

@ianb
Copy link
Contributor

ianb commented Feb 28, 2017

We get the Sentry DSN from the server, but that doesn't let us catch early errors. We should have a default DSN hardcoded into the add-on, but what DSN depends on whether it is dev, stage, live. As a result we need to configure it somehow (probably something like write it to webextension/build/defaultDsn.js)
@ckprice ckprice added this to the Page Shot in 54 milestone Mar 1, 2017
@fzzzy
Copy link
Contributor

fzzzy commented Mar 1, 2017

At build time, if the dsn is configured, it should just get written into that file.

@ianb ianb self-assigned this Mar 3, 2017
dannycoates added a commit that referenced this issue Mar 6, 2017
@dannycoates
Merge pull request #2277 from mozilla-services/default-dsn
129e80d 
Fix #2240, set a default sentryPublicDSN directly in the addon
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ianb ianb

Labels
None yet
Projects
None yet
Milestone
General Release 55
Development

No branches or pull requests
3 participants
@ianb @fzzzy @ckprice