Skip to content

Conversation

vcarl
Copy link
Member

@vcarl vcarl commented Aug 8, 2025

I had observed that no production events were being ingested into Sentry, which is definitely not correct. This should resolve it, and will log configuration details so we can inspect what's happening

vcarl added 2 commits August 8, 2025 13:21
Ran into a situation where I believe that a required env var is not being passed in, and this should help resolve it
Copy link

what-the-diff bot commented Aug 8, 2025

PR Summary

  • GitHub Workflow Configuration
    An update to add new confidential variables SENTRY_INGEST and SENTRY_RELEASES to node.js operations in our automated system.
  • Server Environment Enhancement
    The new confidential variables mentioned above are now retrievable through two newly introduced constants (sentryIngest and sentryReleases) in the server setup file.
  • Error Tracking System Configuration
    The setup process for our error tracking system (Sentry) has been updated. It now uses the sentryIngest constant for key tasks, and all of its options are neatly packaged in a dedicated object. There's also a new log statement added to confirm the system's configuration.
  • Secured Cookie Organization
    The system determining the security level of cookies has been improved by replacing direct reference to the environment variable with the use of a method that checks whether it's a production environment or not. This should offer enhanced clarity and maintain consistency.

@vcarl vcarl merged commit 13e453f into main Aug 8, 2025
5 checks passed
@vcarl vcarl deleted the vc-monitoring branch August 8, 2025 17:34
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.

1 participant