search

AarKro / Hera

Hera - A Discord Chatbot
MIT License
9 stars 1 forks source link

Rework errorhandling #268

Open Pacmens opened 3 years ago

Pacmens commented 3 years ago

As of right now there is a lot of stuff that can outright crash hera.

A lot of these cases there propably exists a better way of handling this.

So the goal should be to rework the error handling to actually handle some possible breaking points instead of just passing the exceptions to the top level.

Pacmens commented 3 years ago

For now this issue is mostly ment to be a collecting container to document cases where more proper errorhandling would make sense.

It will be added to priority as soon as there is enough cases to justify the workload of reviewing everything and redefining the workflows.

Pacmens commented 3 years ago

returning null if database failed. may be better to return an error. could if nothing else make a better stacktrace if deciding to throw something linked to the failedafterretryexception