Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sentry cloudflare #4487

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Sentry cloudflare #4487

wants to merge 2 commits into from

Conversation

rakyi
Copy link
Contributor

@rakyi rakyi commented Jan 24, 2025

I set the SENTRY_DSN in staging wrangler .dev.vars and in the CF dashboard as secrets for both production and preview. See the testing issue.

Resolves #4187

@owidbot
Copy link
Contributor

owidbot commented Jan 24, 2025

Quick links (staging server):

Site Dev Site Preview Admin Wizard Docs

Login: ssh owid@staging-site-sentry-cloudflare

SVG tester:

Number of differences (default views): 0 ✅
Number of differences (all views): 0 ✅

Edited: 2025-01-24 14:36:22 UTC
Execution time: 1.22 seconds

@rakyi rakyi force-pushed the sentry-cloudflare branch 5 times, most recently from 6e91c66 to 7a63d30 Compare January 24, 2025 16:51
@rakyi rakyi marked this pull request as ready for review January 24, 2025 16:53
@rakyi rakyi requested a review from ikesau January 24, 2025 16:53
@rakyi rakyi force-pushed the sentry-cloudflare branch from 7a63d30 to f0e5a70 Compare January 24, 2025 16:54
@rakyi rakyi force-pushed the sentry-cloudflare branch from f0e5a70 to 4b30263 Compare January 24, 2025 16:55
@@ -15,7 +21,7 @@ GRAPHER_CONFIG_R2_BUCKET_FALLBACK_PATH = "v1"
[env.preview.vars]
MAILGUN_DOMAIN = "mg.ourworldindata.org"
SLACK_ERROR_CHANNEL_ID = "C016H0BNNB1"
ENV = "preview"
ENV = "staging"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be consistent with other ENV settings.

* We added the latest @sentry/cloudflare so this is to avoid duplicate
  transitive packages with different versions.
* @sentry/vite-plugin is independent of that, but we might update it
  just as well.
@rakyi rakyi force-pushed the sentry-cloudflare branch from 924b190 to cdc679f Compare January 24, 2025 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cloudflare Function error logging to Sentry
2 participants