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

[Feature] Throw exceptions instead of logging errors manually #770

Open
IceToast opened this issue Mar 24, 2023 · 0 comments
Open

[Feature] Throw exceptions instead of logging errors manually #770

IceToast opened this issue Mar 24, 2023 · 0 comments
Labels
Feature Adding a new feature, or substantial improvements on existing functionality Medium Priority Needs attention, not urgent

Comments

@IceToast
Copy link
Collaborator

Feature Description

At the moment whenever an error occurs while using controlpanel we usually just log the error in the error log and may or may not return the correct http code. Usually it's just a HTTP 500 which does say nothing.

Solution Idea

We should create and use custom exceptions to be thrown when specific errors happen. That would not only make debugging easier but would also help users to fix their own issues.

This is also a way better strategy when implementing (and using) the controlpanel api.

Additional Info

No response

@IceToast IceToast added Feature Adding a new feature, or substantial improvements on existing functionality Medium Priority Needs attention, not urgent labels Mar 24, 2023
@1day2die 1day2die removed this from CPGG Roadmap May 12, 2023
@GeckoBoy84 GeckoBoy84 moved this to 🔖 Ready in CPGG Roadmap May 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Adding a new feature, or substantial improvements on existing functionality Medium Priority Needs attention, not urgent
Projects
Status: 🔖 To Implement
Development

No branches or pull requests

1 participant