diff --git a/build.gradle b/build.gradle index 01c240b5da..ee0c340306 100644 --- a/build.gradle +++ b/build.gradle @@ -359,7 +359,7 @@ project("spring-tx") { compile(project(":spring-beans")) compile(project(":spring-core")) optional(project(":spring-aop")) - optional(project(":spring-context")) // for JCA, @EnableTransactionManagement + optional(project(":spring-context")) // for JCA, @EnableTransactionManagement optional("aopalliance:aopalliance:1.0") optional("org.apache.geronimo.specs:geronimo-jta_1.1_spec:1.1") optional("javax.resource:connector-api:1.5") @@ -434,7 +434,7 @@ project("spring-jdbc") { compile(project(":spring-beans")) compile(project(":spring-core")) compile(project(":spring-tx")) - optional(project(":spring-context")) // for JndiDataSourceLookup + optional(project(":spring-context")) // for JndiDataSourceLookup optional("org.apache.geronimo.specs:geronimo-jta_1.1_spec:1.1") optional("c3p0:c3p0:0.9.1.2") optional("hsqldb:hsqldb:${hsqldbVersion}") @@ -452,8 +452,8 @@ project("spring-context-support") { compile(project(":spring-beans")) compile(project(":spring-context")) provided("javax.activation:activation:1.1") - optional(project(":spring-jdbc")) // for Quartz support - optional(project(":spring-tx")) // for Quartz support + optional(project(":spring-jdbc")) // for Quartz support + optional(project(":spring-tx")) // for Quartz support optional("javax.mail:mail:1.4.7") optional("javax.cache:cache-api:0.5") optional("net.sf.ehcache:ehcache-core:2.0.1") @@ -464,8 +464,8 @@ project("spring-context-support") { optional("com.lowagie:itext:2.1.7") optional("net.sf.jasperreports:jasperreports:3.7.6") testCompile("org.apache.poi:poi:3.6") - testCompile("commons-beanutils:commons-beanutils:1.8.0") // for Velocity/JasperReports - testCompile("commons-digester:commons-digester:1.8.1") // for Velocity/JasperReports + testCompile("commons-beanutils:commons-beanutils:1.8.0") // for Velocity/JasperReports + testCompile("commons-digester:commons-digester:1.8.1") // for Velocity/JasperReports testCompile("hsqldb:hsqldb:${hsqldbVersion}") testCompile("org.slf4j:slf4j-api:${slf4jVersion}") } @@ -478,13 +478,13 @@ project("spring-web") { description = "Spring Web" dependencies { - compile(project(":spring-aop")) // for JaxWsPortProxyFactoryBean - compile(project(":spring-beans")) // for MultiPartFilter + compile(project(":spring-aop")) // for JaxWsPortProxyFactoryBean + compile(project(":spring-beans")) // for MultipartFilter compile(project(":spring-context")) compile(project(":spring-core")) provided("javax.servlet:javax.servlet-api:3.0.1") provided("javax.activation:activation:1.1") - optional(project(":spring-oxm")) // for MarshallingHttpMessageConverter + optional(project(":spring-oxm")) // for MarshallingHttpMessageConverter optional("javax.servlet.jsp:jsp-api:2.1") optional("javax.portlet:portlet-api:2.0") optional("javax.el:el-api:1.0") @@ -578,8 +578,8 @@ project("spring-webmvc") { compile(project(":spring-expression")) compile(project(":spring-web")) provided("javax.servlet:javax.servlet-api:3.0.1") - optional(project(":spring-context-support")) // for Velocity support - optional(project(":spring-oxm")) // for MarshallingView + optional(project(":spring-context-support")) // for Velocity support + optional(project(":spring-oxm")) // for MarshallingView optional("javax.servlet.jsp:jsp-api:2.1") optional("javax.servlet:jstl:1.2") optional("net.sourceforge.jexcelapi:jxl:2.6.12") @@ -786,13 +786,13 @@ project("spring-aspects") { rt("org.aspectj:aspectjrt:${aspectjVersion}") compile("org.aspectj:aspectjweaver:${aspectjVersion}") provided("javax.persistence:persistence-api:1.0") - optional(project(":spring-aop")) // for @Async support - optional(project(":spring-beans")) // for @Configurable support - optional(project(":spring-context")) // for @Enable* support - optional(project(":spring-context-support")) // for JavaMail support - optional(project(":spring-orm")) // for JPA exception translation support - optional(project(":spring-tx")) // for JPA, @Transactional support - testCompile(project(":spring-core")) // for CodeStyleAspect + optional(project(":spring-aop")) // for @Async support + optional(project(":spring-beans")) // for @Configurable support + optional(project(":spring-context")) // for @Enable* support + optional(project(":spring-context-support")) // for JavaMail support + optional(project(":spring-orm")) // for JPA exception translation support + optional(project(":spring-tx")) // for JPA, @Transactional support + testCompile(project(":spring-core")) // for CodeStyleAspect testCompile(project(":spring-test")) testCompile("javax.mail:mail:1.4.7") } diff --git a/spring-beans/src/test/java/org/springframework/beans/factory/DefaultListableBeanFactoryTests.java b/spring-beans/src/test/java/org/springframework/beans/factory/DefaultListableBeanFactoryTests.java index 8ea5f7890c..1dbc07c1c2 100644 --- a/spring-beans/src/test/java/org/springframework/beans/factory/DefaultListableBeanFactoryTests.java +++ b/spring-beans/src/test/java/org/springframework/beans/factory/DefaultListableBeanFactoryTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2013 the original author or authors. + * Copyright 2002-2014 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -104,9 +104,11 @@ public class DefaultListableBeanFactoryTests { private static final Log factoryLog = LogFactory.getLog(DefaultListableBeanFactory.class); + @Rule public ExpectedException thrown = ExpectedException.none(); + @Test public void testUnreferencedSingletonWasInstantiated() { KnowsIfInstantiated.clearInstantiationRecord(); @@ -2676,6 +2678,7 @@ public class DefaultListableBeanFactoryTests { } } + @SuppressWarnings("unused") private static class KnowsIfInstantiated { diff --git a/spring-beans/src/test/resources/org/springframework/beans/factory/xml/testUtilNamespace.xml b/spring-beans/src/test/resources/org/springframework/beans/factory/xml/testUtilNamespace.xml index 2262cd038e..2b71bca48f 100644 --- a/spring-beans/src/test/resources/org/springframework/beans/factory/xml/testUtilNamespace.xml +++ b/spring-beans/src/test/resources/org/springframework/beans/factory/xml/testUtilNamespace.xml @@ -15,7 +15,7 @@ + "/> diff --git a/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToArrayConverter.java b/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToArrayConverter.java index b2098494e8..bb5c7331fc 100644 --- a/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToArrayConverter.java +++ b/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToArrayConverter.java @@ -27,8 +27,9 @@ import org.springframework.core.convert.converter.ConditionalGenericConverter; import org.springframework.util.ObjectUtils; /** - * Converts an Array to another Array. First adapts the source array to a List, then - * delegates to {@link CollectionToArrayConverter} to perform the target array conversion. + * Converts an array to another array. First adapts the source array to a List, + * then delegates to {@link CollectionToArrayConverter} to perform the target + * array conversion. * * @author Keith Donald * @author Phillip Webb diff --git a/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToCollectionConverter.java b/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToCollectionConverter.java index 4e028b49a9..d034eddbca 100644 --- a/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToCollectionConverter.java +++ b/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToCollectionConverter.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2011 the original author or authors. + * Copyright 2002-2014 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -27,23 +27,27 @@ import org.springframework.core.convert.TypeDescriptor; import org.springframework.core.convert.converter.ConditionalGenericConverter; /** - * Converts an Array to a Collection. + * Converts an array to a Collection. * - *

First, creates a new Collection of the requested targetType. + *

First, creates a new Collection of the requested target type. * Then adds each array element to the target collection. - * Will perform an element conversion from the source component type to the collection's parameterized type if necessary. + * Will perform an element conversion from the source component type + * to the collection's parameterized type if necessary. * * @author Keith Donald + * @author Juergen Hoeller * @since 3.0 */ final class ArrayToCollectionConverter implements ConditionalGenericConverter { private final ConversionService conversionService; + public ArrayToCollectionConverter(ConversionService conversionService) { this.conversionService = conversionService; } + public Set getConvertibleTypes() { return Collections.singleton(new ConvertiblePair(Object[].class, Collection.class)); } @@ -58,9 +62,12 @@ final class ArrayToCollectionConverter implements ConditionalGenericConverter { if (source == null) { return null; } + int length = Array.getLength(source); + TypeDescriptor elementDesc = targetType.getElementTypeDescriptor(); Collection target = CollectionFactory.createCollection(targetType.getType(), length); - if (targetType.getElementTypeDescriptor() == null) { + + if (elementDesc == null) { for (int i = 0; i < length; i++) { Object sourceElement = Array.get(source, i); target.add(sourceElement); @@ -70,7 +77,7 @@ final class ArrayToCollectionConverter implements ConditionalGenericConverter { for (int i = 0; i < length; i++) { Object sourceElement = Array.get(source, i); Object targetElement = this.conversionService.convert(sourceElement, - sourceType.elementTypeDescriptor(sourceElement), targetType.getElementTypeDescriptor()); + sourceType.elementTypeDescriptor(sourceElement), elementDesc); target.add(targetElement); } } diff --git a/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToObjectConverter.java b/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToObjectConverter.java index 1db3b2ec7f..f100458a11 100644 --- a/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToObjectConverter.java +++ b/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToObjectConverter.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2011 the original author or authors. + * Copyright 2002-2014 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -25,7 +25,8 @@ import org.springframework.core.convert.TypeDescriptor; import org.springframework.core.convert.converter.ConditionalGenericConverter; /** - * Converts an Array to an Object by returning the first array element after converting it to the desired targetType. + * Converts an array to an Object by returning the first array element + * after converting it to the desired target type. * * @author Keith Donald * @since 3.0 @@ -34,10 +35,12 @@ final class ArrayToObjectConverter implements ConditionalGenericConverter { private final ConversionService conversionService; + public ArrayToObjectConverter(ConversionService conversionService) { this.conversionService = conversionService; } + public Set getConvertibleTypes() { return Collections.singleton(new ConvertiblePair(Object[].class, Object.class)); } diff --git a/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToStringConverter.java b/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToStringConverter.java index 7ffdb86264..1307b991cc 100644 --- a/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToStringConverter.java +++ b/spring-core/src/main/java/org/springframework/core/convert/support/ArrayToStringConverter.java @@ -26,9 +26,9 @@ import org.springframework.core.convert.converter.ConditionalGenericConverter; import org.springframework.util.ObjectUtils; /** - * Converts an Array to a comma-delimited String. - * This implementation first adapts the source Array to a List, - * then delegates to {@link CollectionToStringConverter} to perform the target String conversion. + * Converts an array to a comma-delimited String. First adapts the source array + * to a List, then delegates to {@link CollectionToStringConverter} to perform + * the target String conversion. * * @author Keith Donald * @since 3.0 diff --git a/spring-core/src/main/java/org/springframework/core/convert/support/MapToMapConverter.java b/spring-core/src/main/java/org/springframework/core/convert/support/MapToMapConverter.java index c9f7f9a57f..98b37deaf6 100644 --- a/spring-core/src/main/java/org/springframework/core/convert/support/MapToMapConverter.java +++ b/spring-core/src/main/java/org/springframework/core/convert/support/MapToMapConverter.java @@ -36,6 +36,7 @@ import org.springframework.core.convert.converter.ConditionalGenericConverter; * map's parameterized types K,V if necessary. * * @author Keith Donald + * @author Juergen Hoeller * @since 3.0 */ final class MapToMapConverter implements ConditionalGenericConverter { @@ -61,17 +62,22 @@ final class MapToMapConverter implements ConditionalGenericConverter { if (source == null) { return null; } - boolean copyRequired = !targetType.getType().isInstance(source); Map sourceMap = (Map) source; + + // Shortcut if possible... + boolean copyRequired = !targetType.getType().isInstance(source); if (!copyRequired && sourceMap.isEmpty()) { return sourceMap; } + TypeDescriptor keyDesc = targetType.getMapKeyTypeDescriptor(); + TypeDescriptor valueDesc = targetType.getMapValueTypeDescriptor(); + List targetEntries = new ArrayList(sourceMap.size()); for (Map.Entry entry : sourceMap.entrySet()) { Object sourceKey = entry.getKey(); Object sourceValue = entry.getValue(); - Object targetKey = convertKey(sourceKey, sourceType, targetType.getMapKeyTypeDescriptor()); - Object targetValue = convertValue(sourceValue, sourceType, targetType.getMapValueTypeDescriptor()); + Object targetKey = convertKey(sourceKey, sourceType, keyDesc); + Object targetValue = convertValue(sourceValue, sourceType, valueDesc); targetEntries.add(new MapEntry(targetKey, targetValue)); if (sourceKey != targetKey || sourceValue != targetValue) { copyRequired = true; @@ -80,6 +86,7 @@ final class MapToMapConverter implements ConditionalGenericConverter { if (!copyRequired) { return sourceMap; } + Map targetMap = CollectionFactory.createMap(targetType.getType(), sourceMap.size()); for (MapEntry entry : targetEntries) { entry.addToMap(targetMap); diff --git a/spring-core/src/main/java/org/springframework/core/convert/support/ObjectToArrayConverter.java b/spring-core/src/main/java/org/springframework/core/convert/support/ObjectToArrayConverter.java index 2e8d850d71..9cdbb56228 100644 --- a/spring-core/src/main/java/org/springframework/core/convert/support/ObjectToArrayConverter.java +++ b/spring-core/src/main/java/org/springframework/core/convert/support/ObjectToArrayConverter.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2011 the original author or authors. + * Copyright 2002-2014 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -25,8 +25,8 @@ import org.springframework.core.convert.TypeDescriptor; import org.springframework.core.convert.converter.ConditionalGenericConverter; /** - * Converts an Object to a single-element Array containing the Object. - * Will convert the Object to the target Array's component type if necessary. + * Converts an Object to a single-element array containing the Object. + * Will convert the Object to the target array's component type if necessary. * * @author Keith Donald * @since 3.0 @@ -35,10 +35,12 @@ final class ObjectToArrayConverter implements ConditionalGenericConverter { private final ConversionService conversionService; + public ObjectToArrayConverter(ConversionService conversionService) { this.conversionService = conversionService; } + public Set getConvertibleTypes() { return Collections.singleton(new ConvertiblePair(Object.class, Object[].class)); } diff --git a/spring-core/src/main/java/org/springframework/core/convert/support/ObjectToCollectionConverter.java b/spring-core/src/main/java/org/springframework/core/convert/support/ObjectToCollectionConverter.java index 7741128b53..7890cdb191 100644 --- a/spring-core/src/main/java/org/springframework/core/convert/support/ObjectToCollectionConverter.java +++ b/spring-core/src/main/java/org/springframework/core/convert/support/ObjectToCollectionConverter.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2011 the original author or authors. + * Copyright 2002-2014 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -37,10 +37,12 @@ final class ObjectToCollectionConverter implements ConditionalGenericConverter { private final ConversionService conversionService; + public ObjectToCollectionConverter(ConversionService conversionService) { this.conversionService = conversionService; } + public Set getConvertibleTypes() { return Collections.singleton(new ConvertiblePair(Object.class, Collection.class)); } @@ -54,12 +56,15 @@ final class ObjectToCollectionConverter implements ConditionalGenericConverter { if (source == null) { return null; } + + TypeDescriptor elementDesc = targetType.getElementTypeDescriptor(); Collection target = CollectionFactory.createCollection(targetType.getType(), 1); - if (targetType.getElementTypeDescriptor() == null || targetType.getElementTypeDescriptor().isCollection()) { + + if (elementDesc == null || elementDesc.isCollection()) { target.add(source); } else { - Object singleElement = this.conversionService.convert(source, sourceType, targetType.getElementTypeDescriptor()); + Object singleElement = this.conversionService.convert(source, sourceType, elementDesc); target.add(singleElement); } return target; diff --git a/spring-core/src/main/java/org/springframework/core/convert/support/StringToCollectionConverter.java b/spring-core/src/main/java/org/springframework/core/convert/support/StringToCollectionConverter.java index 74f912c404..b16c246213 100644 --- a/spring-core/src/main/java/org/springframework/core/convert/support/StringToCollectionConverter.java +++ b/spring-core/src/main/java/org/springframework/core/convert/support/StringToCollectionConverter.java @@ -32,6 +32,7 @@ import org.springframework.util.StringUtils; * {@code String.class} can be converted to it. * * @author Keith Donald + * @author Juergen Hoeller * @since 3.0 */ final class StringToCollectionConverter implements ConditionalGenericConverter { @@ -59,16 +60,19 @@ final class StringToCollectionConverter implements ConditionalGenericConverter { return null; } String string = (String) source; + String[] fields = StringUtils.commaDelimitedListToStringArray(string); + TypeDescriptor elementDesc = targetType.getElementTypeDescriptor(); Collection target = CollectionFactory.createCollection(targetType.getType(), fields.length); - if (targetType.getElementTypeDescriptor() == null) { + + if (elementDesc == null) { for (String field : fields) { target.add(field.trim()); } } else { for (String field : fields) { - Object targetElement = this.conversionService.convert(field.trim(), sourceType, targetType.getElementTypeDescriptor()); + Object targetElement = this.conversionService.convert(field.trim(), sourceType, elementDesc); target.add(targetElement); } } diff --git a/spring-core/src/main/java/org/springframework/core/convert/support/StringToEnumConverterFactory.java b/spring-core/src/main/java/org/springframework/core/convert/support/StringToEnumConverterFactory.java index d22ed96b5d..e015afb39b 100644 --- a/spring-core/src/main/java/org/springframework/core/convert/support/StringToEnumConverterFactory.java +++ b/spring-core/src/main/java/org/springframework/core/convert/support/StringToEnumConverterFactory.java @@ -20,12 +20,12 @@ import org.springframework.core.convert.converter.Converter; import org.springframework.core.convert.converter.ConverterFactory; /** - * Converts from a String to a java.lang.Enum by calling {@link Enum#valueOf(Class, String)}. + * Converts from a String to a {@link java.lang.Enum} by calling {@link Enum#valueOf(Class, String)}. * * @author Keith Donald * @since 3.0 */ -@SuppressWarnings({ "unchecked", "rawtypes" }) +@SuppressWarnings({"unchecked", "rawtypes"}) final class StringToEnumConverterFactory implements ConverterFactory { public Converter getConverter(Class targetType) { @@ -34,7 +34,8 @@ final class StringToEnumConverterFactory implements ConverterFactorySubclasses should override the {@code beforeRequest(HttpServletRequest, String)} and - * {@code afterRequest(HttpServletRequest, String)} methods to perform the actual logging around the request. + * {@code afterRequest(HttpServletRequest, String)} methods to perform the actual logging + * around the request. * *

Subclasses are passed the message to write to the log in the {@code beforeRequest} and - * {@code afterRequest} methods. By default, only the URI of the request is logged. However, setting the - * {@code includeQueryString} property to {@code true} will cause the query string of the request to be - * included also. The payload (body) of the request can be logged via the {@code includePayload} flag. Note that - * this will only log that which is read, which might not be the entire payload. + * {@code afterRequest} methods. By default, only the URI of the request is logged. However, + * setting the {@code includeQueryString} property to {@code true} will cause the query string + * of the request to be included also. The payload (body) of the request can be logged via the + * {@code includePayload} flag. Note that this will only log that which is read, which might + * not be the entire payload. * *

Prefixes and suffixes for the before and after messages can be configured using the * {@code beforeMessagePrefix}, {@code afterMessagePrefix}, {@code beforeMessageSuffix} and @@ -87,41 +90,43 @@ public abstract class AbstractRequestLoggingFilter extends OncePerRequestFilter /** - * Set whether or not the query string should be included in the log message.

Should be configured using an - * {@code <init-param>} for parameter name "includeQueryString" in the filter definition in - * {@code web.xml}. + * Set whether the query string should be included in the log message. + *

Should be configured using an {@code <init-param>} for parameter name + * "includeQueryString" in the filter definition in {@code web.xml}. */ public void setIncludeQueryString(boolean includeQueryString) { this.includeQueryString = includeQueryString; } /** - * Return whether or not the query string should be included in the log message. + * Return whether the query string should be included in the log message. */ protected boolean isIncludeQueryString() { return this.includeQueryString; } /** - * Set whether or not the client address and session id should be included in the log message.

Should be configured - * using an {@code <init-param>} for parameter name "includeClientInfo" in the filter definition in - * {@code web.xml}. + * Set whether the client address and session id should be included in the + * log message. + *

Should be configured using an {@code <init-param>} for parameter name + * "includeClientInfo" in the filter definition in {@code web.xml}. */ public void setIncludeClientInfo(boolean includeClientInfo) { this.includeClientInfo = includeClientInfo; } /** - * Return whether or not the client address and session id should be included in the log message. + * Return whether the client address and session id should be included in the + * log message. */ protected boolean isIncludeClientInfo() { return this.includeClientInfo; } /** - * Set whether or not the request payload (body) should be included in the log message.

Should be configured using - * an {@code <init-param>} for parameter name "includePayload" in the filter definition in - * {@code web.xml}. + * Set whether the request payload (body) should be included in the log message. + *

Should be configured using an {@code <init-param>} for parameter name + * "includePayload" in the filter definition in {@code web.xml}. */ public void setIncludePayload(boolean includePayload) { @@ -129,14 +134,15 @@ public abstract class AbstractRequestLoggingFilter extends OncePerRequestFilter } /** - * Return whether or not the request payload (body) should be included in the log message. + * Return whether the request payload (body) should be included in the log message. */ protected boolean isIncludePayload() { - return includePayload; + return this.includePayload; } /** - * Sets the maximum length of the payload body to be included in the log message. Default is 50 characters. + * Sets the maximum length of the payload body to be included in the log message. + * Default is 50 characters. */ public void setMaxPayloadLength(int maxPayloadLength) { Assert.isTrue(maxPayloadLength >= 0, "'maxPayloadLength' should be larger than or equal to 0"); @@ -147,32 +153,36 @@ public abstract class AbstractRequestLoggingFilter extends OncePerRequestFilter * Return the maximum length of the payload body to be included in the log message. */ protected int getMaxPayloadLength() { - return maxPayloadLength; + return this.maxPayloadLength; } /** - * Set the value that should be prepended to the log message written before a request is processed. + * Set the value that should be prepended to the log message written + * before a request is processed. */ public void setBeforeMessagePrefix(String beforeMessagePrefix) { this.beforeMessagePrefix = beforeMessagePrefix; } /** - * Set the value that should be apppended to the log message written before a request is processed. + * Set the value that should be appended to the log message written + * before a request is processed. */ public void setBeforeMessageSuffix(String beforeMessageSuffix) { this.beforeMessageSuffix = beforeMessageSuffix; } /** - * Set the value that should be prepended to the log message written after a request is processed. + * Set the value that should be prepended to the log message written + * after a request is processed. */ public void setAfterMessagePrefix(String afterMessagePrefix) { this.afterMessagePrefix = afterMessagePrefix; } /** - * Set the value that should be appended to the log message written after a request is processed. + * Set the value that should be appended to the log message written + * after a request is processed. */ public void setAfterMessageSuffix(String afterMessageSuffix) { this.afterMessageSuffix = afterMessageSuffix; @@ -200,22 +210,21 @@ public abstract class AbstractRequestLoggingFilter extends OncePerRequestFilter throws ServletException, IOException { boolean isFirstRequest = !isAsyncDispatch(request); + HttpServletRequest requestToUse = request; - if (isIncludePayload()) { - if (isFirstRequest) { - request = new RequestCachingRequestWrapper(request); - } + if (isIncludePayload() && isFirstRequest) { + requestToUse = new RequestCachingRequestWrapper(request); } if (isFirstRequest) { - beforeRequest(request, getBeforeMessage(request)); + beforeRequest(requestToUse, getBeforeMessage(requestToUse)); } try { - filterChain.doFilter(request, response); + filterChain.doFilter(requestToUse, response); } finally { - if (!isAsyncStarted(request)) { - afterRequest(request, getAfterMessage(request)); + if (!isAsyncStarted(requestToUse)) { + afterRequest(requestToUse, getAfterMessage(requestToUse)); } } } @@ -355,9 +364,7 @@ public abstract class AbstractRequestLoggingFilter extends OncePerRequestFilter } return ch; } - } - } }