Spring/spring-framework
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Null-safe handling of response type in AcceptHeaderRequestCallback

Browse Source 
Issue: SPR-16690
See gh-22401
This commit is contained in:
Juergen Hoeller
2018-04-27 18:24:59 +02:00
committed by Brian Clozel
parent a0d420686b
commit 305e4535d8
1 changed files with 6 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
spring-web/src/main/java/org/springframework/web/client/RestTemplate.java
View File

@@ -797,10 +797,8 @@ public class RestTemplate extends InterceptingHttpAccessor implements RestOperat
@Override
public void doWithRequest(ClientHttpRequest request) throws IOException {
if (this.responseType != null) {
final Class<?> responseClass = (this.responseType instanceof Class) ?
(Class<?>) this.responseType : null;
final List<MediaType> allSupportedMediaTypes = getMessageConverters().stream()
.filter(converter -> canReadResponse(responseClass, converter))
List<MediaType> allSupportedMediaTypes = getMessageConverters().stream()
.filter(converter -> canReadResponse(this.responseType, converter))
.flatMap(this::getSupportedMediaTypes)
.distinct()
.sorted(MediaType.SPECIFICITY_COMPARATOR)
@@ -812,15 +810,14 @@ public class RestTemplate extends InterceptingHttpAccessor implements RestOperat
}
}
private boolean canReadResponse(@Nullable Class<?> responseClass, HttpMessageConverter<?> converter) {
private boolean canReadResponse(Type responseType, HttpMessageConverter<?> converter) {
Class<?> responseClass = (responseType instanceof Class ? (Class<?>) responseType : null);
if (responseClass != null) {
return converter.canRead(responseClass, null);
}
else if (converter instanceof GenericHttpMessageConverter) {
GenericHttpMessageConverter<?> genericConverter =
(GenericHttpMessageConverter<?>) converter;
return genericConverter
.canRead(this.responseType, null, null);
GenericHttpMessageConverter<?> genericConverter = (GenericHttpMessageConverter<?>) converter;
return genericConverter.canRead(responseType, null, null);
}
return false;
}