Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: adjust error handling #261

Merged
merged 1 commit into from
Feb 13, 2025
Merged

fix: adjust error handling #261

merged 1 commit into from
Feb 13, 2025

Conversation

Phil91
Copy link
Member

@Phil91 Phil91 commented Feb 13, 2025

Description

fixing the current error handling

Why

after the net 9 upgrade the general error handling was broken and always returned a 500

Issue

N/A

Checklist

  • I have followed the contributing guidelines
  • I have performed IP checks for added or updated 3rd party libraries
  • I have created and linked IP issues or requested their creation by a committer
  • I have performed a self-review of my own code
  • I have successfully tested my changes locally
  • I have added tests that prove my changes work
  • I have checked that new and existing tests pass locally with my changes
  • I have commented my code, particularly in hard-to-understand areas
  • I have added copyright and license headers, footers (for .md files) or files (for images)

@Phil91 Phil91 requested a review from ntruchsess February 13, 2025 10:43
@ntruchsess ntruchsess merged commit 6391eee into main Feb 13, 2025
12 checks passed
@ntruchsess ntruchsess deleted the bugfix/error-handling branch February 13, 2025 12:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: USER READY
Development

Successfully merging this pull request may close these issues.

2 participants