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

Introduce comprehensive error handling strategy #786

Closed
gregturn opened this issue Jan 15, 2019 · 1 comment
Closed

Introduce comprehensive error handling strategy #786

gregturn opened this issue Jan 15, 2019 · 1 comment
Assignees
Labels
in: mediatypes Media type related functionality type: enhancement
Milestone

Comments

@gregturn
Copy link
Contributor

Currently there are few approaches to error handling. The only real thing is VndError, which very HAL derivative.

Having a neutral format that is translated into the mediatype's error format is non-existent.

@gregturn gregturn self-assigned this Aug 30, 2019
@gregturn gregturn added in: mediatypes Media type related functionality type: enhancement labels Aug 30, 2019
@gregturn gregturn modified the milestones: 1.0.0.RC2, 1.0.0.RELEASE Aug 30, 2019
@gregturn gregturn modified the milestones: 1.0.0.RELEASE, 1.0.0.RC3 Sep 16, 2019
@gregturn gregturn modified the milestones: 1.0.0.RC3, 1.0.0.RELEASE Sep 17, 2019
@gregturn gregturn modified the milestones: 1.0.0.RELEASE, 1.1.0.M1 Sep 26, 2019
@gregturn
Copy link
Contributor Author

Resolved via 20bff3e.

VndErrors and VndError were deprecated via a084686, with recommendation to use Problem.

odrotbohm added a commit that referenced this issue Jan 13, 2020
Problem has been redesigned for immutability. Extensions to the payload body are now implemented by wrapping a either an object exposing the properties to add in turn or a plain Map.

Added infrastructure configuration to make sure that Problem instances returned from controller methods (even if wrapped into a ResponseEntity<?>) cause the ContentType header to be set to application/problem+json. HyperMediaMappingInformation now allows to customize the root domain type to signal serialization support for.

Few cleanups regarding the ObjectMapper setup in test cases.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: mediatypes Media type related functionality type: enhancement
Projects
None yet
Development

No branches or pull requests

1 participant