Skip to content

Commit 9fdb1dd

Browse files
committed
Don't log full request when logging channel, close AsyncHttpClient#880
1 parent 1157657 commit 9fdb1dd

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

src/main/java/com/ning/http/client/providers/netty/request/NettyConnectListener.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,10 @@ private void abortChannelPreemption(String partition) {
6565

6666
private void writeRequest(Channel channel) {
6767

68-
LOGGER.debug("Request using non cached Channel '{}':\n{}\n", channel, future.getNettyRequest().getHttpRequest());
68+
LOGGER.debug("Using non-cached Channel {} for {} '{}'",
69+
channel,
70+
future.getNettyRequest().getHttpRequest().getMethod(),
71+
future.getNettyRequest().getHttpRequest().getUri());
6972

7073
Channels.setAttribute(channel, future);
7174

src/main/java/com/ning/http/client/providers/netty/request/NettyRequestSender.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,10 @@ private <T> ListenableFuture<T> sendRequestWithCachedChannel(Request request, Ur
205205
future.setState(NettyResponseFuture.STATE.POOLED);
206206
future.attachChannel(channel, false);
207207

208-
LOGGER.debug("Using cached Channel {}\n for request \n{}\n", channel, future.getNettyRequest().getHttpRequest());
208+
LOGGER.debug("Using cached Channel {} for {} '{}'",
209+
channel,
210+
future.getNettyRequest().getHttpRequest().getMethod(),
211+
future.getNettyRequest().getHttpRequest().getUri());
209212

210213
if (Channels.isChannelValid(channel)) {
211214
Channels.setAttribute(channel, future);

0 commit comments

Comments
 (0)