Hi Michael, Jessie,

Can I please get a code review for CR 6659779. This trivially adds logger invocations to log tunnel requests, ie. HTTP CONNECT.

-Chris.
--- old/src/share/classes/sun/net/www/protocol/http/HttpURLConnection.java      
Tue Apr 15 13:59:44 2008
+++ new/src/share/classes/sun/net/www/protocol/http/HttpURLConnection.java      
Tue Apr 15 13:59:43 2008
@@ -1450,6 +1450,9 @@
                 // so ProgressSource is null.
                 http.parseHTTP(responses, null, this);
 
+                /* Log the response to the CONNECT */
+                logger.fine(responses.toString());
+
                 statusLine = responses.getValue(0);
                 StringTokenizer st = new StringTokenizer(statusLine);
                 st.nextToken();
@@ -1563,6 +1566,10 @@
         requests.setIfNotSet("Accept", acceptString);
 
         setPreemptiveProxyAuthentication(requests);
+
+         /* Log the CONNECT request */
+        logger.fine(requests.toString());
+
         http.writeRequests(requests, null);
         // remove CONNECT header
         requests.set(0, null, null);

Reply via email to