Skip to content

chore(site): add sentry to astro's server config#299

Merged
daniel-hayes merged 1 commit intomainfrom
dh/add-sentry
Jan 8, 2026
Merged

chore(site): add sentry to astro's server config#299
daniel-hayes merged 1 commit intomainfrom
dh/add-sentry

Conversation

@daniel-hayes
Copy link
Copy Markdown
Collaborator

@daniel-hayes daniel-hayes commented Jan 8, 2026

Summary

  • Added Sentry error tracking to the Video.js v10 site
  • Configured Sentry integration for server-side error monitoring

Changes

  1. Added @sentry/astro dependency
  2. Configured Sentry integration (site/astro.config.mjs)
    • Added Sentry integration with project configuration
    • Enabled for production builds with Vercel environment variables
  3. Created server config (site/sentry.server.config.ts)
    • Configured DSN, environment detection, and release tracking
    • Enabled sendDefaultPii for detailed request context
    • Only active in production builds

@vercel
Copy link
Copy Markdown

vercel bot commented Jan 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
vjs-10-demo-html Ready Ready Preview, Comment Jan 8, 2026 2:14am
vjs-10-demo-next Ready Ready Preview, Comment Jan 8, 2026 2:14am
vjs-10-demo-react Ready Ready Preview, Comment Jan 8, 2026 2:14am
vjs-10-website Ready Ready Preview, Comment Jan 8, 2026 2:14am

@daniel-hayes daniel-hayes marked this pull request as ready for review January 8, 2026 02:20
@daniel-hayes daniel-hayes requested a review from decepulis January 8, 2026 02:20
Copy link
Copy Markdown
Collaborator

@decepulis decepulis left a comment

Choose a reason for hiding this comment

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

:chefkiss:

@daniel-hayes daniel-hayes merged commit 644aa30 into main Jan 8, 2026
8 checks passed
@daniel-hayes daniel-hayes deleted the dh/add-sentry branch January 8, 2026 15:06
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.

2 participants