add more comments

Author: justcoding121

Titanium.Web.Proxy/Http/Request.cs

@@ -12,10 +12,24 @@ namespace Titanium.Web.Proxy.Http
     /// </summary>
     public class Request
     {
+        /// <summary>
+        /// Request Method
+        /// </summary>
         public string Method { get; set; }
+
+        /// <summary>
+        /// Request HTTP Uri
+        /// </summary>
         public Uri RequestUri { get; set; }
+
+        /// <summary>
+        /// Request Http Version
+        /// </summary>
         public Version HttpVersion { get; set; }
 
+        /// <summary>
+        /// Request Http hostanem
+        /// </summary>
         internal string Host
         {
             get
@@ -35,6 +49,9 @@ internal string Host
             }
         }
 
+        /// <summary>
+        /// Request content encoding
+        /// </summary>
         internal string ContentEncoding
         {
             get
@@ -50,6 +67,9 @@ internal string ContentEncoding
             }
         }
 
+        /// <summary>
+        /// Request content-length
+        /// </summary>
         public long ContentLength
         {
             get
@@ -68,7 +88,6 @@ public long ContentLength
             }
             set
             {
-
                 var header = RequestHeaders.FirstOrDefault(x => x.Name.ToLower() == "content-length");
 
                 if (value >= 0)
@@ -88,6 +107,9 @@ public long ContentLength
             }
         }
 
+        /// <summary>
+        /// Request content-type
+        /// </summary>
         public string ContentType
         {
             get
@@ -109,6 +131,9 @@ public string ContentType
 
         }
 
+        /// <summary>
+        /// Is request body send as chunked bytes
+        /// </summary>
         public bool IsChunked
         {
             get
@@ -140,6 +165,9 @@ public bool IsChunked
             }
         }
 
+        /// <summary>
+        /// Does this request has a 100-continue header?
+        /// </summary>
         public bool ExpectContinue
         {
             get
@@ -150,19 +178,37 @@ public bool ExpectContinue
             }
         }
 
+        /// <summary>
+        /// Request Url
+        /// </summary>
         public string Url { get { return RequestUri.OriginalString; } }
 
+        /// <summary>
+        /// Encoding for this request
+        /// </summary>
         internal Encoding Encoding { get { return this.GetEncoding(); } }
         /// <summary>
         /// Terminates the underlying Tcp Connection to client after current request
         /// </summary>
         internal bool CancelRequest { get; set; }
 
+        /// <summary>
+        /// Request body as byte array
+        /// </summary>
         internal byte[] RequestBody { get; set; }
+
+        /// <summary>
+        /// request body as string
+        /// </summary>
         internal string RequestBodyString { get; set; }
+
+
         internal bool RequestBodyRead { get; set; }
         internal bool RequestLocked { get; set; }
 
+        /// <summary>
+        /// Does this request has an upgrade to websocket header?
+        /// </summary>
         internal bool UpgradeToWebSocket
         {
             get
@@ -179,8 +225,19 @@ internal bool UpgradeToWebSocket
             }
         }
 
+        /// <summary>
+        /// Request heade collection
+        /// </summary>
         public List<HttpHeader> RequestHeaders { get; set; }
+
+        /// <summary>
+        /// Does server responsed positively for 100 continue request
+        /// </summary>
         public bool Is100Continue { get; internal set; }
+
+        /// <summary>
+        /// Server responsed negatively for the request for 100 continue
+        /// </summary>
         public bool ExpectationFailed { get; internal set; }
 
         public Request()

Titanium.Web.Proxy/Http/Response.cs

@@ -18,7 +18,9 @@ public class Response
 
         internal Encoding Encoding { get { return this.GetResponseCharacterEncoding(); } }
 
-        
+        /// <summary>
+        /// Content encoding for this response
+        /// </summary>
         internal string ContentEncoding
         {
             get
@@ -35,6 +37,10 @@ internal string ContentEncoding
         }
 
         internal Version HttpVersion { get; set; }
+
+        /// <summary>
+        /// Keep the connection alive?
+        /// </summary>
         internal bool ResponseKeepAlive
         {
             get
@@ -51,6 +57,9 @@ internal bool ResponseKeepAlive
             }
         }
 
+        /// <summary>
+        /// Content type of this response
+        /// </summary>
         public string ContentType
         {
             get
@@ -67,6 +76,9 @@ public string ContentType
             }
         }
 
+        /// <summary>
+        /// Length of response body
+        /// </summary>
         internal long ContentLength
         {
             get
@@ -105,6 +117,9 @@ internal long ContentLength
             }
         }
 
+        /// <summary>
+        /// Response transfer-encoding is chunked?
+        /// </summary>
         internal bool IsChunked
         {
             get
@@ -143,14 +158,37 @@ internal bool IsChunked
             }
         }
 
+        /// <summary>
+        /// Collection of all response headers
+        /// </summary>
         public List<HttpHeader> ResponseHeaders { get; set; }
 
+        /// <summary>
+        /// Response network stream
+        /// </summary>
         internal Stream ResponseStream { get; set; }
+
+        /// <summary>
+        /// response body contenst as byte array
+        /// </summary>
         internal byte[] ResponseBody { get; set; }
+
+        /// <summary>
+        /// response body as string
+        /// </summary>
         internal string ResponseBodyString { get; set; }
+
         internal bool ResponseBodyRead { get; set; }
         internal bool ResponseLocked { get; set; }
+
+        /// <summary>
+        /// Is response 100-continue
+        /// </summary>
         public bool Is100Continue { get; internal set; }
+
+        /// <summary>
+        /// expectation failed returned by server?
+        /// </summary>
         public bool ExpectationFailed { get; internal set; }
 
         public Response()

Titanium.Web.Proxy/Network/TcpConnectionManager.cs

@@ -44,7 +44,6 @@ internal static async Task<TcpConnection> GetClient(string hostname, int port, b
             //Get a unique string to identify this connection
             var key = GetConnectionKey(hostname, port, isHttps, version);
 
-
             while (true)
             {
                 await connectionAccessLock.WaitAsync();
@@ -76,6 +75,7 @@ internal static async Task<TcpConnection> GetClient(string hostname, int port, b
 
             }
 
+
             if (cached == null)
                 cached = await CreateClient(hostname, port, isHttps, version);
 

Titanium.Web.Proxy/ProxyServer.cs

@@ -351,7 +351,7 @@ private static void OnAcceptConnection(IAsyncResult asyn)
             }
             catch
             {
-                //Other errors are discarded to keep the proxy running
+                //Other errors are discarded to keep proxy running
             }
 
         }