Browse Source

Merge pull request #1650 from izeye:polish-20180124

* pr/1650:
  Polish "Remove an unnecessary intermediate variable"
  Remove an unnecessary intermediate variable
pull/1650/merge
Stephane Nicoll 8 years ago
parent
commit
6db1b692b8
  1. 9
      spring-web/src/main/java/org/springframework/web/filter/AbstractRequestLoggingFilter.java

9
spring-web/src/main/java/org/springframework/web/filter/AbstractRequestLoggingFilter.java

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2017 the original author or authors. * Copyright 2002-2018 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -327,20 +327,19 @@ public abstract class AbstractRequestLoggingFilter extends OncePerRequestFilter
protected String getMessagePayload(HttpServletRequest request) { protected String getMessagePayload(HttpServletRequest request) {
ContentCachingRequestWrapper wrapper = ContentCachingRequestWrapper wrapper =
WebUtils.getNativeRequest(request, ContentCachingRequestWrapper.class); WebUtils.getNativeRequest(request, ContentCachingRequestWrapper.class);
String payload = null;
if (wrapper != null) { if (wrapper != null) {
byte[] buf = wrapper.getContentAsByteArray(); byte[] buf = wrapper.getContentAsByteArray();
if (buf.length > 0) { if (buf.length > 0) {
int length = Math.min(buf.length, getMaxPayloadLength()); int length = Math.min(buf.length, getMaxPayloadLength());
try { try {
payload = new String(buf, 0, length, wrapper.getCharacterEncoding()); return new String(buf, 0, length, wrapper.getCharacterEncoding());
} }
catch (UnsupportedEncodingException ex) { catch (UnsupportedEncodingException ex) {
payload = "[unknown]"; return "[unknown]";
} }
} }
} }
return payload; return null;
} }

Loading…
Cancel
Save