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

Improve global error handling #16

Merged
merged 13 commits into from
Apr 25, 2024
Merged

Conversation

davidkallesen
Copy link
Contributor

@davidkallesen davidkallesen commented Apr 24, 2024

  • fix: Set statuscode to BadRequest when BadHttpRequestException
  • feat: Add GlobalErrorHandlingOptions
  • feat: Extend GlobalErrorHandlingMiddleware with GlobalErrorHandlingOptions
  • feat: Add WebApplicationExtensions with UseGlobalErrorHandler with optional use of GlobalErrorHandlingOptions
  • chore: Nuget updates

@davidkallesen davidkallesen requested a review from perkops April 24, 2024 21:48

/// <summary>
/// Provides extension methods for the <see cref="WebApplication"/> class to enhance its functionality.
/// This class specifically includes methods for adding global error handling capabilities to the application.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would remove this line, or we need to maintain it all the time :)

@perkops perkops merged commit b447359 into main Apr 25, 2024
4 checks passed
@perkops perkops deleted the feature/Improve-GlobalErrorHandling branch April 25, 2024 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants