Merge pull request #2651 from kapilash/AutoDecompression

Merged. Thanks all.

Author: kapilash

src/Common/src/Interop/Unix/libcurl/Interop.libcurl_types.cs

@@ -22,6 +22,7 @@ internal static partial class CURLoption
             internal const int CURLOPT_WRITEDATA = CurlOptionObjectPointBase + 1;
             internal const int CURLOPT_PROXY = CurlOptionObjectPointBase + 4;
             internal const int CURLOPT_PROXYUSERPWD = CurlOptionObjectPointBase + 6;
+            internal const int CURLOPT_ACCEPTENCODING = CurlOptionObjectPointBase + 102;
         }
 
         // Class for constants defined for the enum CURLINFO in curl.h

src/System.Net.Http/src/System/Net/Http/Unix/CurlHandler.cs

@@ -5,6 +5,7 @@
 using System.Diagnostics;
 using System.Net.Http;
 using System.Runtime.InteropServices;
+using System.Text;
 using System.Threading;
 using System.Threading.Tasks;
 
@@ -21,6 +22,8 @@ internal class CurlHandler : HttpMessageHandler
         #region Constants
 
         private const string UriSchemeHttps = "https";
+        private const string EncodingNameGzip = "gzip";
+        private const string EncodingNameDeflate = "deflate";
         private readonly static string[] AuthenticationSchemes = { "Negotiate", "Digest", "Basic" }; // the order in which libcurl goes over authentication schemes
         #endregion
 
@@ -32,6 +35,7 @@ internal class CurlHandler : HttpMessageHandler
         private IWebProxy _proxy = null;
         private ICredentials _serverCredentials = null;
         private ProxyUsePolicy _proxyPolicy = ProxyUsePolicy.UseDefaultProxy;
+        private DecompressionMethods _automaticDecompression = DecompressionMethods.GZip | DecompressionMethods.Deflate;
 
         #endregion
 
@@ -123,6 +127,27 @@ internal ClientCertificateOption ClientCertificateOptions
             }
         }
 
+        internal bool SupportsAutomaticDecompression
+        {
+            get
+            {
+                return true;
+            }
+        }
+
+        internal DecompressionMethods AutomaticDecompression
+        {
+            get
+            {
+                return _automaticDecompression;
+            }
+            set
+            {
+                CheckDisposedOrStarted();
+                _automaticDecompression = value;
+            }
+        }
+
         #endregion
 
         protected override void Dispose(bool disposing)
@@ -245,11 +270,26 @@ private SafeCurlHandle CreateRequestHandle(RequestCompletionSource state)
             }
 
             SetProxyOptions(state, requestHandle);
+            SetRequestHandleDecompressionOptions(requestHandle);
             // TODO: Handle headers and other options
 
             return requestHandle;
         }
 
+        private void SetRequestHandleDecompressionOptions(SafeCurlHandle requestHandle)
+        {
+            bool gzip = (AutomaticDecompression & DecompressionMethods.GZip) != 0;
+            bool deflate = (AutomaticDecompression & DecompressionMethods.Deflate) != 0;
+            if (gzip || deflate)
+            {
+                string encoding = (gzip && deflate) ?
+                                   EncodingNameGzip + "," + EncodingNameDeflate :
+                                   gzip ? EncodingNameGzip :
+                                   EncodingNameDeflate ;
+                Interop.libcurl.curl_easy_setopt(requestHandle, CURLoption.CURLOPT_ACCEPTENCODING, encoding);
+            }
+        }
+
         private static void SetProxyOptions(RequestCompletionSource state, SafeCurlHandle requestHandle)
         {
             var requestUri = state.RequestMessage.RequestUri;

src/System.Net.Http/src/System/Net/Http/Unix/HttpClientHandler.Unix.cs

@@ -12,7 +12,7 @@ public class HttpClientHandler : HttpMessageHandler
 
         public virtual bool SupportsAutomaticDecompression
         {
-            get { throw NotImplemented.ByDesignWithMessage("HTTP stack not implemented"); }
+            get { return _curlHandler.SupportsAutomaticDecompression; }
         }
 
         public virtual bool SupportsProxy
@@ -51,8 +51,8 @@ public ClientCertificateOption ClientCertificateOptions
 
         public DecompressionMethods AutomaticDecompression
         {
-            get { throw NotImplemented.ByDesignWithMessage("HTTP stack not implemented"); }
-            set { throw NotImplemented.ByDesignWithMessage("HTTP stack not implemented"); }
+            get { return _curlHandler.AutomaticDecompression; }
+            set { _curlHandler.AutomaticDecompression = value; }
         }
 
         public bool UseProxy