Merge pull request #5473 from raydf/devel

Only remove scheme when there's a TLS request ("https")
nim-lang · Mar 3, 2017 · 52462f0 · 52462f0
2 parents 5d125ca + d5d975c
commit 52462f0
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/pure/httpclient.nim b/lib/pure/httpclient.nim
@@ -719,9 +719,9 @@ proc generateHeaders(requestUrl: Uri, httpMethod: string,
     if requestUrl.query.len > 0:
       result.add("?" & requestUrl.query)
   else:
-    # Remove the 'http://' from the URL for CONNECT requests.
+    # Remove the 'http://' from the URL for CONNECT requests for TLS connections.
     var modifiedUrl = requestUrl
-    modifiedUrl.scheme = ""
+    if requestUrl.scheme == "https": modifiedUrl.scheme = ""
     result.add($modifiedUrl)
 
   # HTTP/1.1\c\l