Skip to content

Treat plain ArgumentException as server fault#340

Merged
cdcavell merged 4 commits into
mainfrom
issue-338-argument-exception-mapping
Jul 6, 2026
Merged

Treat plain ArgumentException as server fault#340
cdcavell merged 4 commits into
mainfrom
issue-338-argument-exception-mapping

Conversation

@cdcavell

@cdcavell cdcavell commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Closes #338

Summary

  • Remove the broad ArgumentException to HTTP 400 mapping.
  • Keep BadHttpRequestException mapped to HTTP 400.
  • Update Problem Details tests so plain ArgumentException maps to HTTP 500.
  • Add production detail-hiding coverage for request-level and internal exception paths.

Testing

  • Not run locally; validate with repository CI.

@cdcavell cdcavell merged commit 3410527 into main Jul 6, 2026
8 checks passed
@cdcavell cdcavell deleted the issue-338-argument-exception-mapping branch July 6, 2026 18:49
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.

Review ProblemDetailsExceptionHandler mapping for standard ArgumentException

1 participant