Skip to content

Complete Sentry integration#1546

Merged
charlesBochet merged 1 commit intomainfrom
cbo-fix-sentry
Sep 11, 2023
Merged

Complete Sentry integration#1546
charlesBochet merged 1 commit intomainfrom
cbo-fix-sentry

Conversation

@charlesBochet
Copy link
Copy Markdown
Member

@charlesBochet charlesBochet commented Sep 11, 2023

This PR is a follow up PR on #1403

It specifically:

  • add environment variable validation
  • add log level support that can be controlled from environment variables
  • use captureMessage instead of addBreadcrump in Sentry sdk api (addBreadCrumb was not working for me)
image

I've tested that execeptions and logs are flowing to sentry, it works!

@charlesBochet charlesBochet merged commit 7621854 into main Sep 11, 2023
@charlesBochet charlesBochet deleted the cbo-fix-sentry branch September 11, 2023 22:07
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