Skip to content
This repository has been archived by the owner on Oct 30, 2023. It is now read-only.

Document objectMapper configuration #71

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tobilarscheid
Copy link

Explain that JsonInclude.Non_Null is only needed until the next release of swagger-jaxrs
Once it is out, we can completly remove the line

Explain that JsonInclude.Non_Null is only needed until the next release of swagger-jaxrs
Once it is out, we can completly remove the line
@tobilarscheid
Copy link
Author

As this functionality is finally merged in the swagger-jaxrs code, can we merge this example please?

@martinraie
Copy link

I assume this commit is not necessary anymore as swagger-api/swagger-core#2106 has been reverted in swagger-api/swagger-core#2349?

This row should still be removed:
environment.getObjectMapper().setSerializationInclusion(JsonInclude.Include.NON_NULL);
but replaced with this:
environment.jersey().register(SwaggerSerializers.class);

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants