Skip to content

update sentry sdk#848

Merged
vigneshhari merged 7 commits into
masterfrom
sainAk/issue844
Jun 19, 2022
Merged

update sentry sdk#848
vigneshhari merged 7 commits into
masterfrom
sainAk/issue844

Conversation

@sainak
Copy link
Copy Markdown
Member

@sainak sainak commented Jun 10, 2022

added SENTRY_ENVIRONMENT to set environment
and SENTRY_TRACES_SAMPLE_RATE to set trace sample rate
ignore django.security.DisallowedHost events

closes #844

@sainak sainak requested a review from a team as a code owner June 10, 2022 18:32
@sainak sainak requested a review from Marmik2003 June 11, 2022 08:48
@vigneshhari
Copy link
Copy Markdown
Member

Try to exclude unrelated changes in the PR, there are settings reconfigurations and some precommit changes in a PR about sentry!.

)
sentry_sdk.init(
dsn=SENTRY_DSN,
environment=env("SENTRY_ENVIRONMENT", default="deployment-unknown"),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@sainak make sure @mathew-alex knows about this change, update all concerned deployment documents as well.

@vigneshhari vigneshhari merged commit 9040fdb into master Jun 19, 2022
@sainak sainak deleted the sainAk/issue844 branch September 6, 2022 11:10
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.

Update sentry sdk and specify deployment environment for each deployment

4 participants