8000 Sentry support? · Issue #600 · drashland/drash · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Sentry support? #600

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.

Sign up for GitHub

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

Jump to bottom
Open
ebebbington opened this issue Jan 26, 2022 · 0 comments
Open

Sentry support? #600

ebebbington opened this issue Jan 26, 2022 · 0 comments

Comments

@ebebbington
Copy link
Member

More of a discussion

I think it'd be great to somehow add sentry support to drash applications. This will show even more so that drash can be used in prod and people can switch our their apps for a drash server

I initially thought of it as a service, but a service can't wrap the lifecycle in a try catch, so my new idea would be to bake it in drash itself, which imo has its benefits and drawbacks:

Benefit

  • Sentry could be used for any app built with drash: api, web app etc

Drawback

  • Could bloat codebase
  • Could require a lot of code, but i guess this could be abstracted into a separate file, we import it and just call a method if an exception is thrown and the onfig opt is set
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@crookse @ebebbington
0