Polishing

Author: jhoeller

spring-web/src/main/java/org/springframework/web/accept/MappingMediaTypeFileExtensionResolver.java

@@ -57,7 +57,7 @@ public MappingMediaTypeFileExtensionResolver(@Nullable Map<String, MediaType> me
 			mediaTypes.forEach((extension, mediaType) -> {
 				String lowerCaseExtension = extension.toLowerCase(Locale.ENGLISH);
 				this.mediaTypes.put(lowerCaseExtension, mediaType);
-				addFileExtension(mediaType, extension);
+				addFileExtension(mediaType, lowerCaseExtension);
 				allFileExtensions.add(lowerCaseExtension);
 			});
 			this.allFileExtensions.addAll(allFileExtensions);

spring-web/src/main/java/org/springframework/web/context/request/async/WebAsyncManager.java

@@ -296,15 +296,19 @@ public void startCallableProcessing(final WebAsyncTask<?> webAsyncTask, Object..
 		final CallableInterceptorChain interceptorChain = new CallableInterceptorChain(interceptors);
 
 		this.asyncWebRequest.addTimeoutHandler(() -> {
-			logger.debug("Async request timeout for " + formatRequestUri());
+			if (logger.isDebugEnabled()) {
+				logger.debug("Async request timeout for " + formatRequestUri());
+			}
 			Object result = interceptorChain.triggerAfterTimeout(this.asyncWebRequest, callable);
 			if (result != CallableProcessingInterceptor.RESULT_NONE) {
 				setConcurrentResultAndDispatch(result);
 			}
 		});
 
 		this.asyncWebRequest.addErrorHandler(ex -> {
-			logger.debug("Async request error for " + formatRequestUri() + ": " + ex);
+			if (logger.isDebugEnabled()) {
+				logger.debug("Async request error for " + formatRequestUri() + ": " + ex);
+			}
 			Object result = interceptorChain.triggerAfterError(this.asyncWebRequest, callable, ex);
 			result = (result != CallableProcessingInterceptor.RESULT_NONE ? result : ex);
 			setConcurrentResultAndDispatch(result);
@@ -339,7 +343,6 @@ public void startCallableProcessing(final WebAsyncTask<?> webAsyncTask, Object..
 		}
 	}
 
-	@SuppressWarnings("ConstantConditions")
 	private void logExecutorWarning() {
 		if (taskExecutorWarning && logger.isWarnEnabled()) {
 			synchronized (DEFAULT_TASK_EXECUTOR) {