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

Add ExceptionMapper for RuntimeJsonException #478

Closed
sleberknight opened this issue Apr 30, 2024 · 1 comment · Fixed by #486
Closed

Add ExceptionMapper for RuntimeJsonException #478

sleberknight opened this issue Apr 30, 2024 · 1 comment · Fixed by #486
Assignees
Labels
new feature A new feature such as a new class, method, package, group of classes, etc.
Milestone

Comments

@sleberknight
Copy link
Member

sleberknight commented Apr 30, 2024

Add an ExceptionMapper that handles kiwi's RuntimeJsonException.

Eventually, we might want to move it directly into kiwi.

Assuming the cause of the RuntimeJsonException is JsonProcessingException, it should behave like the existing JsonProcessingExceptionMapper. This is the expected case. If the cause is not JsonProcessingException then we should just return a 500 response, since we don't know what happened.

@sleberknight sleberknight added the new feature A new feature such as a new class, method, package, group of classes, etc. label Apr 30, 2024
@sleberknight sleberknight added this to the 3.6.0 milestone Apr 30, 2024
@sleberknight sleberknight self-assigned this Apr 30, 2024
@sleberknight
Copy link
Member Author

This blocks #485

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature A new feature such as a new class, method, package, group of classes, etc.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant