From b9037821c10273730f13561d5947c48e6c78b00d Mon Sep 17 00:00:00 2001 From: Israel Colomer Date: Mon, 24 Apr 2017 11:45:51 +0100 Subject: [PATCH] Only log request if trace level is on. --- .../stack/datamill/http/impl/ClientImpl.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/core/src/main/java/foundation/stack/datamill/http/impl/ClientImpl.java b/core/src/main/java/foundation/stack/datamill/http/impl/ClientImpl.java index ec2208e..a527e14 100644 --- a/core/src/main/java/foundation/stack/datamill/http/impl/ClientImpl.java +++ b/core/src/main/java/foundation/stack/datamill/http/impl/ClientImpl.java @@ -91,7 +91,7 @@ public Observable request( HttpUriRequest request = buildHttpRequest(method, targetURI); setRequestOptions(request, options); setRequestHeaders(request, headers); - printRequestIfDebugging(method, targetURI, headers); + printRequestIfTracingLog(method, targetURI, headers); CloseableHttpResponse httpResponse; if (body != null) { @@ -156,13 +156,13 @@ protected CloseableHttpResponse doExecute(CloseableHttpClient httpClient, HttpUr return httpClient.execute(request); } - private void printRequestIfDebugging(Method method, URI composedUri, Multimap headers) { - if (logger.isDebugEnabled()) { - logger.debug("Making HTTP request {} {}", method.name(), composedUri); + private void printRequestIfTracingLog(Method method, URI composedUri, Multimap headers) { + if (logger.isTraceEnabled()) { + logger.trace("Making HTTP request {} {}", method.name(), composedUri); if (headers != null && logger.isDebugEnabled()) { - logger.debug(" HTTP request headers:"); + logger.trace(" HTTP request headers:"); for (Map.Entry header : headers.entries()) { - logger.debug(" {}: {}", header.getKey(), header.getValue()); + logger.trace(" {}: {}", header.getKey(), header.getValue()); } } }