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

Allow entry point for Sentry #3170

Closed
MatzeKitt opened this issue Feb 27, 2020 · 7 comments
Closed

Allow entry point for Sentry #3170

MatzeKitt opened this issue Feb 27, 2020 · 7 comments

Comments

@MatzeKitt
Copy link

Since the current error logging mechanism of the WoltLab Suite Core is quiet nice, especially with the new email notifications, it lacks some functionality:

  • Real time notifications
  • Notifications via other channels, e. g. Slack
  • There is often no information which user triggered the error, which can be helpful for resolving the problem
  • If the system is really broken, the notification may not being sent or not even logged

To overcome this, it would be really nice to have the possibility to integrate a error logging system like Sentry as soon as possible inside the code. This would resolve the above list of issues and maybe some more I didn’t notice so far.

Why Sentry? It is the de facto standard software for error logging, its free to use – either self-hosted or even the basic version via cloud hosting – and Open Source.

@RouL
Copy link
Contributor

RouL commented Mar 4, 2020

We're using self-hosted sentry at our company and I'm also using it for some private projects. I couldn't agree more that the WoltLab Suite could use this.
Probably also a nice addition for WoltLabs own Cloud service to watch the health of customer instances. ;)

@BurntimeX BurntimeX added this to Needs Triage in WoltLab Suite 5.3 via automation May 23, 2020
@BurntimeX BurntimeX moved this from Needs Triage to Enhancement in WoltLab Suite 5.3 May 23, 2020
@dtdesign dtdesign added this to Enhancement in WoltLab Suite 5.4 Jun 17, 2020
@dtdesign dtdesign removed this from Enhancement in WoltLab Suite 5.3 Jun 17, 2020
@dtdesign dtdesign moved this from Enhancement to New Feature in WoltLab Suite 5.4 Jun 23, 2020
@TimWolla TimWolla removed this from Feature in WoltLab Suite 5.4 Feb 22, 2021
@TimWolla TimWolla added this to Needs Triage in WoltLab Suite 5.5 via automation Feb 22, 2021
@dtdesign dtdesign moved this from Needs Triage to Concept / Needs Work in WoltLab Suite 5.5 Apr 21, 2021
@TimWolla TimWolla removed this from Concept / Needs Work in WoltLab Suite 5.5 Oct 4, 2021
@TimWolla TimWolla added this to Needs Triage in WoltLab Suite 6.0 via automation Oct 4, 2021
@dtdesign dtdesign removed this from Needs Triage in WoltLab Suite 6.0 Mar 6, 2023
@dtdesign dtdesign closed this as completed Mar 6, 2023
@MatzeKitt
Copy link
Author

So it actually won’t happen or did I miss something?

@dtdesign
Copy link
Member

dtdesign commented Mar 6, 2023

It has been on our list for way too long already and there are some fundamental design decision that are blocking a reliable integration of such a service. We do believe that the integration of services like Sentry can be beneficial to some extent, but for various reason this is not a priority for us for the next 1-2 years.

We’ll revisit the integration of such services at a later point in time.

@syntafin
Copy link

syntafin commented Mar 6, 2023

As the time goes on, after 3 years I am not sure if Sentry as wished here would be a good solution.

For me, I dropped Sentry in Personal projects because of performance problems when the capturing was active.

@dtdesign
Copy link
Member

dtdesign commented Mar 6, 2023

“Sentry” is just one answer to a much broader question. We never really focused on that particular product but rather saw it as an example for a class of products that attempt to solve this kind of problems.

@Fighter456
Copy link
Contributor

We’ll revisit the integration of such services at a later point in time.

Apart from the fact that I am no longer interested in it due to the fact that I have turned away from your products, I would like to ask anyway:

If such an integration (whether for Sentry or something else) isn't completely off the table, then why is the issue closed as "completed" (which is totally wrong) instead of leaving it in an open state in the backlog?

@dtdesign
Copy link
Member

dtdesign commented Mar 7, 2023

If such an integration (whether for Sentry or something else) isn't completely off the table, then why is the issue closed as "completed" (which is totally wrong) instead of leaving it in an open state in the backlog?

I’m not sure why it reads “completed” and at the same time this appears to be the result when the issue is simply being closed. I don’t think its possible to change the message.

Regarding the backlog: We’re removing items from the backlog that have been postponed over and over again and it’s clear that we either are unable to take action in the near feature or do not intend to make changes. In this particular case such an integration requires us to make significant architectural changes that isn’t worth the trade-off neither at this point nor in the foreseeable future.

Edit: Oh, there is a drop-down option to select a different reason. Interesting.

@dtdesign dtdesign closed this as not planned Won't fix, can't repro, duplicate, stale Mar 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants