PR Comments

Following PR comments were addressed:
*. curl_version_info_data is defined till the required field
*. the fields were made readonly.

Author: kapilash

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

@@ -163,13 +163,16 @@ internal struct CURLMsg
             internal int result;
         }
 
-        // Curl Version Information
-        [StructLayout(LayoutKind.Explicit)]
+        // NOTE: The definition of this structure in Curl/curl.h is larger than
+        // than what is defined below. This definition is only valid for use with
+        // Marshal.PtrToStructure and not for general use in P/Invoke signatures.
+        [StructLayout(LayoutKind.Sequential)]
         internal struct curl_version_info_data
         {
-            [FieldOffset(0)]
             internal int age;
-            [FieldOffset(32)]
+            private unsafe char *version;
+            private int versionNum;
+            private unsafe char *host;
             internal int features;
         }
 

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

@@ -48,8 +48,8 @@ internal partial class CurlHandler : HttpMessageHandler
 
         #region Fields
 
-        private static bool _supportsAutomaticDecompression;
-        private static bool _supportsSSL;
+        private static readonly bool _supportsAutomaticDecompression;
+        private static readonly bool _supportsSSL;
         private volatile bool _anyOperationStarted;
         private volatile bool _disposed;
         private IWebProxy _proxy = null;