SpringCloud/spring-cloud-gateway
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1251 from Gsealy/2.1.x

Browse Source 
clean duplicate logger in ServerWebExchangeUtils class
This commit is contained in:
Ryan Baxter
2019-08-14 10:40:39 -04:00
committed by GitHub
parent b11fd5b2f9 b0b4c0e375
commit 0046d60ce5
1 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
spring-cloud-gateway-core/src/main/java/org/springframework/cloud/gateway/support/ServerWebExchangeUtils.java
View File

@@ -154,8 +154,6 @@ public final class ServerWebExchangeUtils {
*/
public static final String CACHED_REQUEST_BODY_ATTR = "cachedRequestBody";
private static final Log logger = LogFactory.getLog(ServerWebExchangeUtils.class);
private ServerWebExchangeUtils() {
throw new AssertionError("Must not instantiate utility class.");
}
@@ -175,8 +173,8 @@ public final class ServerWebExchangeUtils {
public static boolean setResponseStatus(ServerWebExchange exchange,
HttpStatus httpStatus) {
boolean response = exchange.getResponse().setStatusCode(httpStatus);
if (!response && logger.isWarnEnabled()) {
logger.warn("Unable to set status code to " + httpStatus
if (!response && log.isWarnEnabled()) {
log.warn("Unable to set status code to " + httpStatus
+ ". Response already committed.");
}
return response;
@@ -187,8 +185,8 @@ public final class ServerWebExchangeUtils {
if (exchange.getResponse().isCommitted()) {
return false;
}
if (logger.isDebugEnabled()) {
logger.debug("Setting response status to " + statusHolder);
if (log.isDebugEnabled()) {
log.debug("Setting response status to " + statusHolder);
}
if (statusHolder.getHttpStatus() != null) {
return setResponseStatus(exchange, statusHolder.getHttpStatus());