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 improvements #241

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Sentry improvements #241

wants to merge 5 commits into from

Conversation

rosemcc
Copy link
Contributor

@rosemcc rosemcc commented Feb 28, 2022

Description

This PR adds a couple of improvements for Sentry tracking, including adding sourcemaps to releases via the CI/CD pipeline, and adding user IPs to enable auto session tracking (need to test that this works - documentation is unclear)

Solution

  • add to sentry initialisation config: initialScope: { user: {ip_address: "{{auto}}"} }, autoSessionTracking: true

  • Modify angular.json to generate sourcemaps in 'hidden' mode, i.e. "sourceMap": { "hidden": true,

  • Modify the Jenkins pipeline to include the following steps:

    • set the build version as the commit SHA, so it can be used as the Sentry release number
    • set the Sentry environment variables (auth token, project name etc)
    • create a sentry Release using the commit SHA and upload the sourcemaps as part of the release
  • Remove the current Sentry release workflow from the .github folder.

Screenshots

N/A

Testing

N/A

Have the changes been checked in the following browsers?

N/A

@@ -3,6 +3,7 @@
export const environment = {
env: 'prod',
sentryTracesSampleRate: 0.2,
version: 'VERSION',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

testing a comment

@rosemcc rosemcc removed the request for review from eric-el-tan March 21, 2022 20:35
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