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);