You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by ol...@apache.org on 2014/08/16 13:06:55 UTC
svn commit: r1618328 -
/httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/client/protocol/ResponseProcessCookies.java
Author: olegk
Date: Sat Aug 16 11:06:55 2014
New Revision: 1618328
URL: http://svn.apache.org/r1618328
Log:
HTTPCLIENT-1544: Fixed NPE in ResponseProcessCookies when running with DEBUG priority
Modified:
httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/client/protocol/ResponseProcessCookies.java
Modified: httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/client/protocol/ResponseProcessCookies.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/client/protocol/ResponseProcessCookies.java?rev=1618328&r1=1618327&r2=1618328&view=diff
==============================================================================
--- httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/client/protocol/ResponseProcessCookies.java (original)
+++ httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/client/protocol/ResponseProcessCookies.java Sat Aug 16 11:06:55 2014
@@ -138,10 +138,12 @@ public class ResponseProcessCookies impl
buf.append(cookie.getName());
buf.append("=\"");
String v = cookie.getValue();
- if (v.length() > 100) {
- v = v.substring(0, 100) + "...";
+ if (v != null) {
+ if (v.length() > 100) {
+ v = v.substring(0, 100) + "...";
+ }
+ buf.append(v);
}
- buf.append(v);
buf.append("\"");
buf.append(", version:");
buf.append(Integer.toString(cookie.getVersion()));