SPR-6303 - Add more logging to RestTemplate

Author: poutsma

org.springframework.web/src/main/java/org/springframework/web/client/RestTemplate.java

@@ -382,8 +382,11 @@ protected <T> T doExecute(URI url,
 				requestCallback.doWithRequest(request);
 			}
 			response = request.execute();
-			if (getErrorHandler().hasError(response)) {
-				getErrorHandler().handleError(response);
+			if (!getErrorHandler().hasError(response)) {
+				logResponseStatus(method, url, response);
+			}
+			else {
+				handleResponseError(method, url, response);
 			}
 			if (responseExtractor != null) {
 				return responseExtractor.extractData(response);
@@ -419,6 +422,31 @@ private void checkForSupportedMessageConverter(Class type) {
 		throw new IllegalArgumentException("Could not resolve HttpMessageConverter for [" + type.getName() + "]");
 	}
 
+	private void logResponseStatus(HttpMethod method, URI url, ClientHttpResponse response) {
+		if (logger.isDebugEnabled()) {
+			try {
+				logger.debug(method.name() + " request for \"" + url + "\" resulted in " + response.getStatusCode() +
+						" (" + response.getStatusText() + ")");
+			}
+			catch (IOException e) {
+				// ignore
+			}
+		}
+	}
+
+	private void handleResponseError(HttpMethod method, URI url, ClientHttpResponse response) throws IOException {
+		if (logger.isWarnEnabled()) {
+			try {
+				logger.warn(method.name() + " request for \"" + url + "\" resulted in " + response.getStatusCode() +
+						" (" + response.getStatusText() + "); invoking error handler");
+			}
+			catch (IOException e) {
+				// ignore
+			}
+		}
+		getErrorHandler().handleError(response);
+	}
+
 	/** Request callback implementation that prepares the request's accept headers. */
 	private class AcceptHeaderRequestCallback<T> implements RequestCallback {