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

feat: Add sentry #36

Merged
merged 1 commit into from
Jun 10, 2024
Merged

feat: Add sentry #36

merged 1 commit into from
Jun 10, 2024

Conversation

evanpurkhiser
Copy link
Member

No description provided.


pub async fn execute() -> io::Result<()> {
Copy link
Member Author

Choose a reason for hiding this comment

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

Per our documentation

In a multithreaded application, spawned threads should inherit the Hub from the main thread. In order for that to happen, the Sentry client must be initialized before starting an async runtime or spawning threads. This means you'll have to avoid using macros such as #[tokio::main] or #[actix_web::main], because they start the runtime first.

@@ -0,0 +1,9 @@
pub fn init() -> () {
let _guard = sentry::init((
"https://[email protected]/4507408549347328",
Copy link
Member Author

Choose a reason for hiding this comment

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

Will read from a config file once I get to it

@evanpurkhiser evanpurkhiser enabled auto-merge (squash) June 10, 2024 21:22
@evanpurkhiser evanpurkhiser merged commit ca52e89 into main Jun 10, 2024
5 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/feat-add-sentry branch June 10, 2024 22:04
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