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

Handle Background Worker's exceptions #30

Open
petrspelos opened this issue Dec 29, 2021 · 0 comments
Open

Handle Background Worker's exceptions #30

petrspelos opened this issue Dec 29, 2021 · 0 comments
Labels
Backend good first issue Good for newcomers help wanted Extra attention is needed maintenance Improvement tasks

Comments

@petrspelos
Copy link
Member

petrspelos commented Dec 29, 2021

Summary

In case the background worker encounters and error, it should output it into its log instead of closing.

Alternatively the log file name could be appended with -failed.log to better recognize this scenario.

A general try/catch could work. Alternatively we can hook into the UnhandledException event of the runtime to be a bit more sure.

The exception should also be printed in the log itself.

Notes

Careful about exceptions that are coming from failed attempt at logging.

Want to work on this issue?

❤️ Amazing! Let us know in the comments bellow so that we can assign this issue to you. 🎉

No stress, though, feel free to ask for help with any clarifications or problems you may encounter along the way. We're here to help.

@petrspelos petrspelos added help wanted Extra attention is needed good first issue Good for newcomers Backend maintenance Improvement tasks labels Dec 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend good first issue Good for newcomers help wanted Extra attention is needed maintenance Improvement tasks
Projects
None yet
Development

No branches or pull requests

1 participant