Improve Host HTTP header (#181)

***NO_CI***
nanoframework · Mar 30, 2021 · 3dbe0d4 · 3dbe0d4
1 parent a159ab6
commit 3dbe0d4
Showing 1 changed file with 2 additions and 17 deletions.
diff --git a/nanoFramework.System.Net.Http/Http/System.Net.HttpWebRequest.cs b/nanoFramework.System.Net.Http/Http/System.Net.HttpWebRequest.cs
@@ -1264,7 +1264,8 @@ private void PrepareHeaders()
                 }
             }
 
-            m_httpRequestHeaders.ChangeInternal(HttpKnownHeaderNames.Host, ConnectHostAndPort());
+            m_httpRequestHeaders.ChangeInternal(HttpKnownHeaderNames.Host, m_originalUrl.Host);
+
             // Adds user name and password for basic Http authentication.
             if (m_NetworkCredentials != null && m_NetworkCredentials.AuthenticationType == AuthenticationType.Basic)
             {   // If credentials are supplied, we need to add header like "Authorization: Basic QWxhZGRpbjpvcGVuIHNlc2FtZQ=="
@@ -1291,22 +1292,6 @@ private void PrepareHeaders()
             m_requestSent = true;
         }
 
-        /// <summary>
-        /// Return string with remote Host and Port if port is not default.
-        /// Need update for HTTPS.
-        /// </summary>
-        /// <returns>String with host Url and port corresponding to target Uri.</returns>
-        internal string ConnectHostAndPort()
-        {
-            string retStr = m_originalUrl.Host;
-            if (m_originalUrl.Port != 80)
-            {
-                retStr += ":" + m_originalUrl.Port;
-            }
-
-            return retStr;
-        }
-
         /// <summary>
         /// Removes the given stream from the connection pool
         /// </summary>