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

Ensure defaultSentryDSN is set in the Firefox tree #2782

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

Ensure defaultSentryDSN is set in the Firefox tree #2782

ianb opened this issue Apr 28, 2017 · 1 comment
Assignees

Comments

@ianb
Copy link
Contributor

ianb commented Apr 28, 2017

My simple attempt to import Screenshots into the tree left browser/extensions/screenshots/webextension/build/buildSettings.js with an empty DSN.

@ghost ghost assigned ianb May 1, 2017
@ghost ghost added this to the Screenshots in 54 milestone May 1, 2017
@ianb
Copy link
Contributor Author

ianb commented May 1, 2017

This is set by the environmental variable $SCREENSHOTS_SENTRY which should set before building. The value for prod is https://d58fc53f92cd47bba7266ad6444514d8@sentry.prod.mozaws.net/74

So this should do it:

$ SCREENSHOTS_SENTRY="https://d58fc53f92cd47bba7266ad6444514d8@sentry.prod.mozaws.net/74" ./bin/export_mc.py ...

However, it looks like export_mc.py has some server settings, it would be great to include these Sentry settings too. So the settings should be:

DEFAULT_SENTRY = {
    "local": "",
    "dev": "https://d58fc53f92cd47bba7266ad6444514d8@sentry.prod.mozaws.net/74",
    "stage": "https://3d2efd7bad9e4e359014cb4e69b8eaec@sentry.prod.mozaws.net/140",
    "prod": "https://97d8afa496f94764ae255e739b147f4b@sentry.prod.mozaws.net/139",
}

@ianb ianb assigned jaredhirsch and unassigned ianb May 1, 2017
@ianb ianb closed this as completed in 576a9c8 May 2, 2017
ianb added a commit that referenced this issue May 2, 2017
Fixes #2782, set Sentry URL when exporting to m-c
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants