Improve logging

Author: gthea

src/main/java/io/split/android/client/network/ProxyCacertConnectionHandler.java

@@ -76,7 +76,6 @@ HttpResponse executeRequest(@NonNull HttpProxy httpProxy,
 
                 // If the origin is HTTPS, wrap the tunnel socket with a new SSLSocket (system CA)
                 if (HTTPS.equalsIgnoreCase(targetUrl.getProtocol())) {
-                    Logger.v("Wrapping tunnel socket with new SSLSocket for origin server handshake");
                     try {
                         // Use the provided SSLSocketFactory, which is configured to trust the origin's CA
                         finalSocket = sslSocketFactory.createSocket(
@@ -99,7 +98,6 @@ HttpResponse executeRequest(@NonNull HttpProxy httpProxy,
                         } else {
                             throw new IOException("Failed to create SSLSocket to origin");
                         }
-                        Logger.v("SSL handshake with origin server completed");
                     } catch (Exception sslEx) {
                         Logger.e("Failed to establish SSL connection to origin: " + sslEx.getMessage());
                         throw new IOException("Failed to establish SSL connection to origin server", sslEx);
@@ -135,6 +133,7 @@ HttpResponse executeRequest(@NonNull HttpProxy httpProxy,
             // Let socket-related IOExceptions pass through unwrapped for consistent error handling
             throw e;
         } catch (Exception e) {
+            Logger.e("Failed to execute request through custom tunnel: " + e.getMessage());
             throw new IOException("Failed to execute request through custom tunnel", e);
         }
     }
@@ -168,7 +167,6 @@ HttpStreamResponse executeStreamRequest(@NonNull HttpProxy httpProxy,
 
                 // If the origin is HTTPS, wrap the tunnel socket with a new SSLSocket (system CA)
                 if (HTTPS.equalsIgnoreCase(targetUrl.getProtocol())) {
-                    Logger.v("Wrapping tunnel socket with new SSLSocket for origin server handshake");
                     try {
                         // Use the provided SSLSocketFactory, which is configured to trust the origin's CA
                         finalSocket = sslSocketFactory.createSocket(
@@ -191,7 +189,6 @@ HttpStreamResponse executeStreamRequest(@NonNull HttpProxy httpProxy,
                         } else {
                             throw new IOException("Failed to create SSLSocket to origin");
                         }
-                        Logger.v("SSL handshake with origin server completed");
                     } catch (Exception sslEx) {
                         Logger.e("Failed to establish SSL connection to origin: " + sslEx.getMessage());
                         throw new IOException("Failed to establish SSL connection to origin server", sslEx);

src/main/java/io/split/android/client/network/RawHttpResponseParser.java

@@ -40,7 +40,6 @@ HttpResponse parseHttpResponse(@NonNull InputStream inputStream, Certificate[] s
             throw new IOException("No HTTP response received from server");
         }
 
-        Logger.v("Parsing HTTP status line: " + statusLine);
         int statusCode = parseStatusCode(statusLine);
 
         // 2. Read and parse response headers directly
@@ -201,7 +200,7 @@ private String readChunkedBodyWithCharset(InputStream inputStream, Charset chars
                 // Read trailing headers until empty line
                 String trailerLine;
                 while ((trailerLine = readLineFromStream(inputStream)) != null && !trailerLine.trim().isEmpty()) {
-                    Logger.v("Chunked trailer: " + trailerLine);
+                    // no-op
                 }
                 break;
             }

src/main/java/io/split/android/client/network/SslProxyTunnelEstablisher.java

@@ -19,8 +19,6 @@
 import javax.net.ssl.SSLSocket;
 import javax.net.ssl.SSLSocketFactory;
 
-import io.split.android.client.utils.logger.Logger;
-
 /**
  * Establishes SSL tunnels to SSL proxies using CONNECT protocol.
  */
@@ -127,8 +125,6 @@ private void sendConnectRequest(@NonNull SSLSocket sslSocket,
                                     int targetPort,
                                     @Nullable ProxyCredentialsProvider proxyCredentialsProvider) throws IOException {
 
-        Logger.v("Sending CONNECT request through SSL: CONNECT " + targetHost + ":" + targetPort + " HTTP/1.1");
-
         PrintWriter writer = new PrintWriter(new OutputStreamWriter(sslSocket.getOutputStream(), StandardCharsets.UTF_8), false);
         writer.write("CONNECT " + targetHost + ":" + targetPort + " HTTP/1.1" + CRLF);
         writer.write("Host: " + targetHost + ":" + targetPort + CRLF);
@@ -140,8 +136,6 @@ private void sendConnectRequest(@NonNull SSLSocket sslSocket,
         // Send empty line to end headers
         writer.write(CRLF);
         writer.flush();
-
-        Logger.v("CONNECT request sent through SSL connection");
     }
 
     private void addProxyAuthHeader(@NonNull ProxyCredentialsProvider proxyCredentialsProvider, PrintWriter writer) {
@@ -167,8 +161,6 @@ private void addProxyAuthHeader(@NonNull ProxyCredentialsProvider proxyCredentia
      */
     private void validateConnectResponse(@NonNull SSLSocket sslSocket) throws IOException {
 
-        Logger.v("Reading CONNECT response through SSL connection");
-
         try {
             BufferedReader reader = new BufferedReader(new InputStreamReader(sslSocket.getInputStream(), StandardCharsets.UTF_8));
 
@@ -177,8 +169,6 @@ private void validateConnectResponse(@NonNull SSLSocket sslSocket) throws IOExce
                 throw new IOException("No CONNECT response received from proxy");
             }
 
-            Logger.v("Received CONNECT response through SSL: " + statusLine.trim());
-
             // Parse status code
             String[] statusParts = statusLine.split(" ");
             if (statusParts.length < 2) {
@@ -195,7 +185,7 @@ private void validateConnectResponse(@NonNull SSLSocket sslSocket) throws IOExce
             // Read headers until empty line (but don't process them for CONNECT)
             String headerLine;
             while ((headerLine = reader.readLine()) != null && !headerLine.trim().isEmpty()) {
-                Logger.v("CONNECT response header: " + headerLine);
+                // no-op
             }
 
             // Check status code