SpringCloud/spring-cloud-stream
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Checkstyle

Browse Source
This commit is contained in:
Oleg Zhurakousky
2022-10-17 17:07:51 +02:00
parent a5aa6d152c
commit 60a78a617d
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
core/spring-cloud-stream/src/main/java/org/springframework/cloud/stream/converter/CompositeMessageConverterFactory.java
View File

@@ -76,7 +76,7 @@ public class CompositeMessageConverterFactory {
List<? extends MessageConverter> customConverters,
ObjectMapper objectMapper, JsonMapper jsonMapper) {
this.objectMapper = objectMapper == null ? new ObjectMapper() : objectMapper;
this.jsonMapper = jsonMapper == null ? new JacksonMapper(objectMapper) : jsonMapper;
this.jsonMapper = jsonMapper == null ? new JacksonMapper(objectMapper) : jsonMapper;
if (!CollectionUtils.isEmpty(customConverters)) {
this.converters = new ArrayList<>(customConverters);
}
@@ -111,7 +111,7 @@ public class CompositeMessageConverterFactory {
@Nullable Object conversionHint) {
/*
* We must revisit this. This is a copy from ApplicationMarshallingMessageConverter which derived from an older class etc. . .
* This attempts to use JSON conversion to convert something that is not json in the first place.
* This attempts to use JSON conversion to convert something that is not json in the first place.
* For example Integer payload with application/json CT should actually fail since Integer is not a JSON.
* This is !!!!wrong!!!!! and ONLY remains here for backward compatibility.
*/