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(errorHandling): adjust general error handler #165

Merged
merged 2 commits into from
Feb 13, 2025
Merged

Conversation

Phil91
Copy link
Collaborator

@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 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 force-pushed the bugfix/error-handling branch from 1ccf3f6 to 3b269e7 Compare February 13, 2025 12:55
Co-authored-by: Norbert Truchsess <norbert.truchsess@t-online.de>
@Phil91 Phil91 merged commit 1c32a68 into main Feb 13, 2025
7 checks passed
@Phil91 Phil91 deleted the bugfix/error-handling branch February 13, 2025 14:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants