WebRequest.DefaultWebProxy: Set doesn't work without a previous Get (#…

…22798) * WebRequest.DefaultWebProxy: Set doesn't work without a previous Get
dotnet · Sep 8, 2017 · fd5a525 · fd5a525
1 parent 3fbb419
commit fd5a525
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/src/System.Net.Requests/src/System/Net/WebRequest.cs b/src/System.Net.Requests/src/System/Net/WebRequest.cs
@@ -573,6 +573,7 @@ public static IWebProxy DefaultWebProxy
                 lock (s_internalSyncObject)
                 {
                     s_DefaultWebProxy = value;
+                    s_DefaultWebProxyInitialized = true;
                 }
             }
         }

diff --git a/src/System.Net.Requests/tests/WebRequestTest.cs b/src/System.Net.Requests/tests/WebRequestTest.cs
@@ -2,11 +2,12 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for more information.
 
+using System.Diagnostics;
 using Xunit;
 
 namespace System.Net.Tests
 {
-    public class WebRequestTest
+    public class WebRequestTest : RemoteExecutorTestBase
     {
         static WebRequestTest()
         {
@@ -28,6 +29,20 @@ public void DefaultWebProxy_VerifyDefaults_Success()
 
             Assert.Null(initialDefaultWebProxyCredentials);
         }
+
+        [Fact]
+        public void DefaultWebProxy_SetThenGet_ValuesMatch()
+        {
+            RemoteInvoke(() =>
+            {
+                IWebProxy p = new WebProxy();
+
+                WebRequest.DefaultWebProxy = p;
+                Assert.Same(p, WebRequest.DefaultWebProxy);
+
+                return SuccessExitCode;
+            }).Dispose();
+        }
 
         [Fact]
         public void DefaultWebProxy_SetCredentialsToNullThenGet_ValuesMatch()