[MVC] RedirectHandler follows HTTP status code 303 See other (#33246)

Missing HTTP status code from the 3xx class was added.

Author: ambrosmiroslav

src/Mvc/Mvc.Testing/src/Handlers/RedirectHandler.cs

@@ -173,6 +173,7 @@ private static bool ShouldKeepVerb(HttpResponseMessage response) =>
         private bool IsRedirect(HttpResponseMessage response) =>
             response.StatusCode == HttpStatusCode.MovedPermanently ||
                 response.StatusCode == HttpStatusCode.Redirect ||
+                response.StatusCode == HttpStatusCode.RedirectMethod ||
                 response.StatusCode == HttpStatusCode.RedirectKeepVerb ||
                 (int)response.StatusCode == 308;
     }

src/Mvc/test/Mvc.FunctionalTests/TestingInfrastructureTests.cs

@@ -114,6 +114,20 @@ public async Task TestingInfrastructure_RedirectHandlerUsesOriginalRequestHeader
             Assert.Equal("false", modifiedHeaderWasSent);
         }
 
+        [Fact]
+        public async Task TestingInfrastructure_RedirectHandlerFollowsStatusCode303()
+        {
+            // Act
+            var request = new HttpRequestMessage(HttpMethod.Get, "Testing/RedirectHandler/Redirect303");
+            var client = Factory.CreateDefaultClient(
+                new RedirectHandler());
+            var response = await client.SendAsync(request);
+
+            // Assert
+            Assert.Equal(HttpStatusCode.OK, response.StatusCode);
+            Assert.Equal("Test", await response.Content.ReadAsStringAsync());
+        }
+
         [Fact]
         public async Task TestingInfrastructure_PostRedirectGetWorksWithCookies()
         {

src/Mvc/test/WebSites/BasicWebSite/Controllers/TestingController.cs

@@ -4,6 +4,7 @@
 using System.Linq;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Mvc;
+using System.Net;
 
 namespace BasicWebSite.Controllers
 {
@@ -63,6 +64,30 @@ public IActionResult RedirectHandlerHeadersRedirect()
             }
         }
 
+        [HttpGet("Testing/RedirectHandler/Redirect303")]
+        public IActionResult RedirectHandlerStatusCode303()
+        {
+            return new RedirectUsingStatusCode("Testing/Builder", HttpStatusCode.SeeOther);
+        }
+
+        public class RedirectUsingStatusCode : ActionResult
+        {
+            private string _url;
+            private HttpStatusCode _statusCode;
+
+            public RedirectUsingStatusCode(string url, HttpStatusCode statusCode)
+            {
+                _url = url;
+                _statusCode = statusCode;
+            }
+
+            public override void ExecuteResult(ActionContext context)
+            {
+                context.HttpContext.Response.Redirect(_url);
+                context.HttpContext.Response.StatusCode = (int)_statusCode;
+            }
+        }
+
         [HttpGet("Testing/AntiforgerySimulator/{value}")]
         public IActionResult AntiforgerySimulator([FromRoute]int value)
         {

src/Mvc/test/WebSites/GenericHostWebSite/Controllers/TestingController.cs

@@ -4,6 +4,7 @@
 using System.Linq;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Mvc;
+using System.Net;
 
 namespace GenericHostWebSite.Controllers
 {
@@ -63,6 +64,30 @@ public IActionResult RedirectHandlerHeadersRedirect()
             }
         }
 
+        [HttpGet("Testing/RedirectHandler/Redirect303")]
+        public IActionResult RedirectHandlerStatusCode303()
+        {
+            return new RedirectUsingStatusCode("Testing/Builder", HttpStatusCode.SeeOther);
+        }
+
+        public class RedirectUsingStatusCode : ActionResult
+        {
+            private string _url;
+            private HttpStatusCode _statusCode;
+
+            public RedirectUsingStatusCode(string url, HttpStatusCode statusCode)
+            {
+                _url = url;
+                _statusCode = statusCode;
+            }
+
+            public override void ExecuteResult(ActionContext context)
+            {
+                context.HttpContext.Response.Redirect(_url);
+                context.HttpContext.Response.StatusCode = (int)_statusCode;
+            }
+        }
+
         [HttpGet("Testing/AntiforgerySimulator/{value}")]
         public IActionResult AntiforgerySimulator([FromRoute]int value)
         {