Add utf8 text overloads for creating results (#42595)

davidfowl · web-flow · commit 6bac4240ec43 · 2022-07-11T19:44:34.000-07:00
* Add utf8 text overloads for creating results
- This change adds overloads of Text on Results and TypedResults that take ReadOnlySpan&lt;byte&gt; for utf8 content.
- Added tests
diff --git a/src/Http/Http.Abstractions/src/HttpResults/IStatusCodeHttpResult.cs b/src/Http/Http.Abstractions/src/HttpResults/IStatusCodeHttpResult.cs
@@ -1,4 +1,4 @@
-﻿// Licensed to the .NET Foundation under one or more agreements.
+// Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
 namespace Microsoft.AspNetCore.Http;
@@ -12,5 +12,5 @@ public interface IStatusCodeHttpResult
     /// <summary>
     /// Gets the HTTP status code.
     /// </summary>
-    int StatusCode { get; }
+    int? StatusCode { get; }
 }
diff --git a/src/Http/Http.Abstractions/src/PublicAPI.Unshipped.txt b/src/Http/Http.Abstractions/src/PublicAPI.Unshipped.txt
@@ -21,7 +21,7 @@ Microsoft.AspNetCore.Http.INestedHttpResult
 Microsoft.AspNetCore.Http.INestedHttpResult.Result.get -> Microsoft.AspNetCore.Http.IResult!
 Microsoft.AspNetCore.Http.IRouteHandlerFilter.InvokeAsync(Microsoft.AspNetCore.Http.RouteHandlerInvocationContext! context, Microsoft.AspNetCore.Http.RouteHandlerFilterDelegate! next) -> System.Threading.Tasks.ValueTask<object?>
 Microsoft.AspNetCore.Http.IStatusCodeHttpResult
-Microsoft.AspNetCore.Http.IStatusCodeHttpResult.StatusCode.get -> int
+Microsoft.AspNetCore.Http.IStatusCodeHttpResult.StatusCode.get -> int?
 Microsoft.AspNetCore.Http.IValueHttpResult
 Microsoft.AspNetCore.Http.IValueHttpResult.Value.get -> object?
 Microsoft.AspNetCore.Http.IValueHttpResult<TValue>
diff --git a/src/Http/Http.Results/src/Accepted.cs b/src/Http/Http.Results/src/Accepted.cs
@@ -46,6 +46,8 @@ internal Accepted(Uri locationUri)
     /// </summary>
     public int StatusCode => StatusCodes.Status202Accepted;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <summary>
     /// Gets the location at which the status of the requested content can be monitored.
     /// </summary>
diff --git a/src/Http/Http.Results/src/AcceptedAtRoute.cs b/src/Http/Http.Results/src/AcceptedAtRoute.cs
@@ -54,6 +54,8 @@ internal AcceptedAtRoute(
     /// </summary>
     public int StatusCode => StatusCodes.Status202Accepted;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/AcceptedAtRouteOfT.cs b/src/Http/Http.Results/src/AcceptedAtRouteOfT.cs
@@ -67,6 +67,8 @@ internal AcceptedAtRoute(
     /// </summary>
     public int StatusCode => StatusCodes.Status202Accepted;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/AcceptedOfT.cs b/src/Http/Http.Results/src/AcceptedOfT.cs
@@ -65,6 +65,8 @@ internal Accepted(Uri locationUri, TValue? value)
     /// </summary>
     public int StatusCode => StatusCodes.Status202Accepted;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <summary>
     /// Gets the location at which the status of the requested content can be monitored.
     /// </summary>
diff --git a/src/Http/Http.Results/src/BadRequest.cs b/src/Http/Http.Results/src/BadRequest.cs
@@ -26,6 +26,8 @@ internal BadRequest()
     /// </summary>
     public int StatusCode => StatusCodes.Status400BadRequest;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/BadRequestOfT.cs b/src/Http/Http.Results/src/BadRequestOfT.cs
@@ -37,6 +37,8 @@ internal BadRequest(TValue? error)
     /// </summary>
     public int StatusCode => StatusCodes.Status400BadRequest;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/Conflict.cs b/src/Http/Http.Results/src/Conflict.cs
@@ -26,6 +26,8 @@ internal Conflict()
     /// </summary>
     public int StatusCode => StatusCodes.Status409Conflict;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/ConflictOfT.cs b/src/Http/Http.Results/src/ConflictOfT.cs
@@ -37,6 +37,8 @@ internal Conflict(TValue? error)
     /// </summary>
     public int StatusCode => StatusCodes.Status409Conflict;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/ContentHttpResult.cs b/src/Http/Http.Results/src/ContentHttpResult.cs
@@ -50,8 +50,6 @@ internal ContentHttpResult(string? content, string? contentType, int? statusCode
     /// </summary>
     public int? StatusCode { get; internal init; }
 
-    int IStatusCodeHttpResult.StatusCode => StatusCode ?? StatusCodes.Status200OK;
-
     /// <summary>
     /// Writes the content to the HTTP response.
     /// </summary>
diff --git a/src/Http/Http.Results/src/Created.cs b/src/Http/Http.Results/src/Created.cs
@@ -50,6 +50,8 @@ internal Created(Uri locationUri)
     /// </summary>
     public int StatusCode => StatusCodes.Status201Created;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public string? Location { get; }
 
diff --git a/src/Http/Http.Results/src/CreatedAtRoute.cs b/src/Http/Http.Results/src/CreatedAtRoute.cs
@@ -54,6 +54,8 @@ internal CreatedAtRoute(
     /// </summary>
     public int StatusCode => StatusCodes.Status201Created;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/CreatedAtRouteOfT.cs b/src/Http/Http.Results/src/CreatedAtRouteOfT.cs
@@ -67,6 +67,8 @@ internal CreatedAtRoute(
     /// </summary>
     public int StatusCode => StatusCodes.Status201Created;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/CreatedOfT.cs b/src/Http/Http.Results/src/CreatedOfT.cs
@@ -65,6 +65,8 @@ internal Created(Uri locationUri, TValue? value)
     /// </summary>
     public int StatusCode => StatusCodes.Status201Created;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public string? Location { get; }
 
diff --git a/src/Http/Http.Results/src/HttpResultsHelper.cs b/src/Http/Http.Results/src/HttpResultsHelper.cs
@@ -13,7 +13,7 @@ namespace Microsoft.AspNetCore.Http;
 
 internal static partial class HttpResultsHelper
 {
-    private const string DefaultContentType = "text/plain; charset=utf-8";
+    internal const string DefaultContentType = "text/plain; charset=utf-8";
     private static readonly Encoding DefaultEncoding = Encoding.UTF8;
 
     public static Task WriteResultAsJsonAsync<T>(
diff --git a/src/Http/Http.Results/src/JsonHttpResultOfT.cs b/src/Http/Http.Results/src/JsonHttpResultOfT.cs
@@ -78,8 +78,6 @@ internal JsonHttpResult(TValue? value, int? statusCode, string? contentType, Jso
     /// </summary>
     public int? StatusCode { get; }
 
-    int IStatusCodeHttpResult.StatusCode => StatusCode ?? StatusCodes.Status200OK;
-
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/NoContent.cs b/src/Http/Http.Results/src/NoContent.cs
@@ -25,6 +25,8 @@ internal NoContent()
     /// </summary>
     public int StatusCode => StatusCodes.Status204NoContent;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/NotFound.cs b/src/Http/Http.Results/src/NotFound.cs
@@ -25,6 +25,8 @@ internal NotFound()
     /// </summary>
     public int StatusCode => StatusCodes.Status404NotFound;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/NotFoundOfT.cs b/src/Http/Http.Results/src/NotFoundOfT.cs
@@ -36,6 +36,8 @@ internal NotFound(TValue? value)
     /// </summary>
     public int StatusCode => StatusCodes.Status404NotFound;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/Ok.cs b/src/Http/Http.Results/src/Ok.cs
@@ -25,6 +25,8 @@ internal Ok()
     /// </summary>
     public int StatusCode => StatusCodes.Status200OK;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/OkOfT.cs b/src/Http/Http.Results/src/OkOfT.cs
@@ -36,6 +36,8 @@ internal Ok(TValue? value)
     /// </summary>
     public int StatusCode => StatusCodes.Status200OK;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/ProblemHttpResult.cs b/src/Http/Http.Results/src/ProblemHttpResult.cs
@@ -43,6 +43,8 @@ internal ProblemHttpResult(ProblemDetails problemDetails)
     /// </summary>
     public int StatusCode => ProblemDetails.Status!.Value;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc/>
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/PublicAPI.Unshipped.txt b/src/Http/Http.Results/src/PublicAPI.Unshipped.txt
@@ -146,6 +146,11 @@ Microsoft.AspNetCore.Http.HttpResults.RedirectToRouteHttpResult.RouteName.get ->
 Microsoft.AspNetCore.Http.HttpResults.RedirectToRouteHttpResult.RouteValues.get -> Microsoft.AspNetCore.Routing.RouteValueDictionary?
 *REMOVED*static Microsoft.AspNetCore.Http.Results.Content(string! content, string? contentType = null, System.Text.Encoding? contentEncoding = null) -> Microsoft.AspNetCore.Http.IResult!
 *REMOVED*static Microsoft.AspNetCore.Http.Results.Content(string! content, Microsoft.Net.Http.Headers.MediaTypeHeaderValue! contentType) -> Microsoft.AspNetCore.Http.IResult!
+Microsoft.AspNetCore.Http.HttpResults.Utf8ContentHttpResult
+Microsoft.AspNetCore.Http.HttpResults.Utf8ContentHttpResult.ContentType.get -> string?
+Microsoft.AspNetCore.Http.HttpResults.Utf8ContentHttpResult.ExecuteAsync(Microsoft.AspNetCore.Http.HttpContext! httpContext) -> System.Threading.Tasks.Task!
+Microsoft.AspNetCore.Http.HttpResults.Utf8ContentHttpResult.ResponseContent.get -> System.ReadOnlyMemory<byte>
+Microsoft.AspNetCore.Http.HttpResults.Utf8ContentHttpResult.StatusCode.get -> int?
 static Microsoft.AspNetCore.Http.Results.Accepted<TValue>(string? uri = null, TValue? value = default(TValue?)) -> Microsoft.AspNetCore.Http.IResult!
 static Microsoft.AspNetCore.Http.Results.AcceptedAtRoute<TValue>(string? routeName = null, object? routeValues = null, TValue? value = default(TValue?)) -> Microsoft.AspNetCore.Http.IResult!
 static Microsoft.AspNetCore.Http.Results.BadRequest<TValue>(TValue? error) -> Microsoft.AspNetCore.Http.IResult!
@@ -162,11 +167,13 @@ static Microsoft.AspNetCore.Http.Results.NotFound<TValue>(TValue? value) -> Micr
 static Microsoft.AspNetCore.Http.Results.Ok<TValue>(TValue? value) -> Microsoft.AspNetCore.Http.IResult!
 static Microsoft.AspNetCore.Http.Results.Text(string? content, string? contentType = null, System.Text.Encoding? contentEncoding = null, int? statusCode = null) -> Microsoft.AspNetCore.Http.IResult!
 static Microsoft.AspNetCore.Http.Results.Text(string? content, string? contentType, System.Text.Encoding? contentEncoding) -> Microsoft.AspNetCore.Http.IResult!
+static Microsoft.AspNetCore.Http.Results.Text(System.ReadOnlySpan<byte> utf8Content, string? contentType = null, int? statusCode = null) -> Microsoft.AspNetCore.Http.IResult!
 static Microsoft.AspNetCore.Http.Results.UnprocessableEntity<TValue>(TValue? error) -> Microsoft.AspNetCore.Http.IResult!
 static Microsoft.AspNetCore.Http.TypedResults.Content(string? content, string? contentType = null, System.Text.Encoding? contentEncoding = null, int? statusCode = null) -> Microsoft.AspNetCore.Http.HttpResults.ContentHttpResult!
 static Microsoft.AspNetCore.Http.TypedResults.Content(string? content, string? contentType, System.Text.Encoding? contentEncoding) -> Microsoft.AspNetCore.Http.HttpResults.ContentHttpResult!
 static Microsoft.AspNetCore.Http.TypedResults.Text(string? content, string? contentType = null, System.Text.Encoding? contentEncoding = null, int? statusCode = null) -> Microsoft.AspNetCore.Http.HttpResults.ContentHttpResult!
 static Microsoft.AspNetCore.Http.TypedResults.Text(string? content, string? contentType, System.Text.Encoding? contentEncoding) -> Microsoft.AspNetCore.Http.HttpResults.ContentHttpResult!
+static Microsoft.AspNetCore.Http.TypedResults.Text(System.ReadOnlySpan<byte> utf8Content, string? contentType = null, int? statusCode = null) -> Microsoft.AspNetCore.Http.HttpResults.Utf8ContentHttpResult!
 ~Microsoft.AspNetCore.Http.HttpResults.Results<TResult1, TResult2, TResult3, TResult4, TResult5, TResult6>
 ~Microsoft.AspNetCore.Http.HttpResults.Results<TResult1, TResult2, TResult3, TResult4, TResult5, TResult6>.ExecuteAsync(Microsoft.AspNetCore.Http.HttpContext httpContext) -> System.Threading.Tasks.Task
 ~Microsoft.AspNetCore.Http.HttpResults.Results<TResult1, TResult2, TResult3, TResult4, TResult5, TResult6>.Result.get -> Microsoft.AspNetCore.Http.IResult
diff --git a/src/Http/Http.Results/src/Results.cs b/src/Http/Http.Results/src/Results.cs
@@ -142,9 +142,23 @@ public static IResult Text(string? content, string? contentType, Encoding? conte
     /// If encoding is provided by both the 'charset' and the <paramref name="contentEncoding"/> parameters, then
     /// the <paramref name="contentEncoding"/> parameter is chosen as the final encoding.
     /// </remarks>
+#pragma warning disable RS0026 // Do not add multiple public overloads with optional parameters
     public static IResult Text(string? content, string? contentType = null, Encoding? contentEncoding = null, int? statusCode = null)
+#pragma warning restore RS0026 // Do not add multiple public overloads with optional parameters
         => TypedResults.Text(content, contentType, contentEncoding, statusCode);
 
+    /// <summary>
+    /// Writes the <paramref name="utf8Content"/> UTF-8 encoded text to the HTTP response.
+    /// </summary>
+    /// <param name="utf8Content">The content to write to the response.</param>
+    /// <param name="contentType">The content type (MIME type).</param>
+    /// <param name="statusCode">The status code to return.</param>
+    /// <returns>The created <see cref="IResult"/> object for the response.</returns>
+#pragma warning disable RS0027 // Public API with optional parameter(s) should have the most parameters amongst its public overloads
+    public static IResult Text(ReadOnlySpan<byte> utf8Content, string? contentType = null, int? statusCode = null)
+#pragma warning restore RS0027 // Public API with optional parameter(s) should have the most parameters amongst its public overloads
+        => TypedResults.Text(utf8Content, contentType, statusCode);
+
     /// <summary>
     /// Writes the <paramref name="content"/> string to the HTTP response.
     /// </summary>
diff --git a/src/Http/Http.Results/src/StatusCodeHttpResult.cs b/src/Http/Http.Results/src/StatusCodeHttpResult.cs
@@ -27,6 +27,8 @@ internal StatusCodeHttpResult(int statusCode)
     /// </summary>
     public int StatusCode { get; }
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <summary>
     /// Sets the status code on the HTTP response.
     /// </summary>
diff --git a/src/Http/Http.Results/src/TypedResults.cs b/src/Http/Http.Results/src/TypedResults.cs
@@ -131,6 +131,18 @@ public static ContentHttpResult Content(string? content, string? contentType = n
     public static ContentHttpResult Text(string? content, string? contentType, Encoding? contentEncoding)
         => Text(content, contentType, contentEncoding, null);
 
+    /// <summary>
+    /// Writes the <paramref name="utf8Content"/> UTF8 text content to the HTTP response.
+    /// </summary>
+    /// <param name="utf8Content">The content to write to the response.</param>
+    /// <param name="contentType">The content type (MIME type).</param>
+    /// <param name="statusCode">The status code to return.</param>
+    /// <returns>The created <see cref="Utf8ContentHttpResult"/> object for the response.</returns>
+#pragma warning disable RS0027 // Public API with optional parameter(s) should have the most parameters amongst its public overloads.
+    public static Utf8ContentHttpResult Text(ReadOnlySpan<byte> utf8Content, string? contentType = null, int? statusCode = null)
+#pragma warning restore RS0027 // Public API with optional parameter(s) should have the most parameters amongst its public overloads.
+        => new Utf8ContentHttpResult(utf8Content, contentType, statusCode);
+
     /// <summary>
     /// Writes the <paramref name="content"/> string to the HTTP response.
     /// <para>
@@ -146,7 +158,9 @@ public static ContentHttpResult Text(string? content, string? contentType, Encod
     /// <param name="contentEncoding">The content encoding.</param>
     /// <param name="statusCode">The status code to return.</param>
     /// <returns>The created <see cref="ContentHttpResult"/> object for the response.</returns>
+#pragma warning disable RS0026 // Do not add multiple public overloads with optional parameters
     public static ContentHttpResult Text(string? content, string? contentType = null, Encoding? contentEncoding = null, int? statusCode = null)
+#pragma warning restore RS0026 // Do not add multiple public overloads with optional parameters
     {
         MediaTypeHeaderValue? mediaTypeHeaderValue = null;
         if (contentType is not null)
diff --git a/src/Http/Http.Results/src/UnauthorizedHttpResult.cs b/src/Http/Http.Results/src/UnauthorizedHttpResult.cs
@@ -24,6 +24,8 @@ internal UnauthorizedHttpResult()
     /// </summary>
     public int StatusCode => StatusCodes.Status401Unauthorized;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc />
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/UnprocessableEntity.cs b/src/Http/Http.Results/src/UnprocessableEntity.cs
@@ -26,6 +26,8 @@ internal UnprocessableEntity()
     /// </summary>
     public int StatusCode => StatusCodes.Status422UnprocessableEntity;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc />
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/UnprocessableEntityOfT.cs b/src/Http/Http.Results/src/UnprocessableEntityOfT.cs
@@ -37,6 +37,8 @@ internal UnprocessableEntity(TValue? value)
     /// </summary>
     public int StatusCode => StatusCodes.Status422UnprocessableEntity;
 
+    int? IStatusCodeHttpResult.StatusCode => StatusCode;
+
     /// <inheritdoc />
     public Task ExecuteAsync(HttpContext httpContext)
     {
diff --git a/src/Http/Http.Results/src/Utf8ContentHttpResult.cs b/src/Http/Http.Results/src/Utf8ContentHttpResult.cs
@@ -0,0 +1,68 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.Logging;
+
+namespace Microsoft.AspNetCore.Http.HttpResults;
+
+/// <summary>
+/// An <see cref="Utf8ContentHttpResult"/> that when executed
+/// will produce a response with content.
+/// </summary>
+public sealed partial class Utf8ContentHttpResult : IResult, IStatusCodeHttpResult, IContentTypeHttpResult
+{
+    /// <summary>
+    /// Initializes a new instance of the <see cref="Utf8ContentHttpResult"/> class with the values
+    /// </summary>
+    /// <param name="utf8Content">The value to format in the entity body.</param>
+    /// <param name="statusCode">The HTTP status code of the response.</param>
+    /// <param name="contentType">The Content-Type header for the response.</param>
+    internal Utf8ContentHttpResult(ReadOnlySpan<byte> utf8Content, string? contentType, int? statusCode)
+    {
+        // We need to make a copy here since we have to stash it on the heap
+        ResponseContent = utf8Content.ToArray();
+        StatusCode = statusCode;
+        ContentType = contentType;
+    }
+
+    /// <summary>
+    /// Gets the content representing the body of the response.
+    /// </summary>
+    public ReadOnlyMemory<byte> ResponseContent { get; internal init; }
+
+    /// <summary>
+    /// Gets the Content-Type header for the response.
+    /// </summary>
+    public string? ContentType { get; internal init; }
+
+    /// <summary>
+    /// Gets the HTTP status code.
+    /// </summary>
+    public int? StatusCode { get; internal init; }
+
+    /// <summary>
+    /// Writes the content to the HTTP response.
+    /// </summary>
+    /// <param name="httpContext">The <see cref="HttpContext"/> for the current request.</param>
+    /// <returns>A task that represents the asynchronous execute operation.</returns>
+    public Task ExecuteAsync(HttpContext httpContext)
+    {
+        ArgumentNullException.ThrowIfNull(httpContext);
+
+        // Creating the logger with a string to preserve the category after the refactoring.
+        var loggerFactory = httpContext.RequestServices.GetRequiredService<ILoggerFactory>();
+        var logger = loggerFactory.CreateLogger("Microsoft.AspNetCore.Http.Result.Utf8ContentHttpResult");
+
+        if (StatusCode is { } statusCode)
+        {
+            HttpResultsHelper.Log.WritingResultAsStatusCode(logger, statusCode);
+            httpContext.Response.StatusCode = statusCode;
+        }
+
+        httpContext.Response.ContentType = ContentType ?? HttpResultsHelper.DefaultContentType;
+
+        httpContext.Response.ContentLength = ResponseContent.Length;
+        return httpContext.Response.Body.WriteAsync(ResponseContent).AsTask();
+    }
+}
diff --git a/src/Http/Http.Results/src/ValidationProblem.cs b/src/Http/Http.Results/src/ValidationProblem.cs
diff --git a/src/Http/Http.Results/test/ContentResultTests.cs b/src/Http/Http.Results/test/ContentResultTests.cs
diff --git a/src/Http/Http.Results/test/JsonResultTests.cs b/src/Http/Http.Results/test/JsonResultTests.cs
diff --git a/src/Http/Http.Results/test/ResultsTests.cs b/src/Http/Http.Results/test/ResultsTests.cs
diff --git a/src/Http/Http.Results/test/TypedResultsTests.cs b/src/Http/Http.Results/test/TypedResultsTests.cs
diff --git a/src/Http/Http.Results/test/Utf8ContentResultTests.cs b/src/Http/Http.Results/test/Utf8ContentResultTests.cs

Original file line number	Diff line number	Diff line change
`@@ -54,6 +54,8 @@ internal AcceptedAtRoute(`
`54`	`54`	`/// </summary>`
`55`	`55`	`public int StatusCode => StatusCodes.Status202Accepted;`
`56`	`56`
	`57`	`+ int? IStatusCodeHttpResult.StatusCode => StatusCode;`
	`58`	`+`
`57`	`59`	`/// <inheritdoc/>`
`58`	`60`	`public Task ExecuteAsync(HttpContext httpContext)`
`59`	`61`	`{`
Original file line number	Diff line number	Diff line change
`@@ -67,6 +67,8 @@ internal AcceptedAtRoute(`
`67`	`67`	`/// </summary>`
`68`	`68`	`public int StatusCode => StatusCodes.Status202Accepted;`
`69`	`69`
	`70`	`+ int? IStatusCodeHttpResult.StatusCode => StatusCode;`
	`71`	`+`
`70`	`72`	`/// <inheritdoc/>`
`71`	`73`	`public Task ExecuteAsync(HttpContext httpContext)`
`72`	`74`	`{`
Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,8 @@ internal BadRequest()`
`26`	`26`	`/// </summary>`
`27`	`27`	`public int StatusCode => StatusCodes.Status400BadRequest;`
`28`	`28`
	`29`	`+ int? IStatusCodeHttpResult.StatusCode => StatusCode;`
	`30`	`+`
`29`	`31`	`/// <inheritdoc/>`
`30`	`32`	`public Task ExecuteAsync(HttpContext httpContext)`
`31`	`33`	`{`
Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,8 @@ internal BadRequest(TValue? error)`
`37`	`37`	`/// </summary>`
`38`	`38`	`public int StatusCode => StatusCodes.Status400BadRequest;`
`39`	`39`
	`40`	`+ int? IStatusCodeHttpResult.StatusCode => StatusCode;`
	`41`	`+`
`40`	`42`	`/// <inheritdoc/>`
`41`	`43`	`public Task ExecuteAsync(HttpContext httpContext)`
`42`	`44`	`{`
Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,8 @@ internal Conflict()`
`26`	`26`	`/// </summary>`
`27`	`27`	`public int StatusCode => StatusCodes.Status409Conflict;`
`28`	`28`
	`29`	`+ int? IStatusCodeHttpResult.StatusCode => StatusCode;`
	`30`	`+`
`29`	`31`	`/// <inheritdoc/>`
`30`	`32`	`public Task ExecuteAsync(HttpContext httpContext)`
`31`	`33`	`{`
Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,8 @@ internal Conflict(TValue? error)`
`37`	`37`	`/// </summary>`
`38`	`38`	`public int StatusCode => StatusCodes.Status409Conflict;`
`39`	`39`
	`40`	`+ int? IStatusCodeHttpResult.StatusCode => StatusCode;`
	`41`	`+`
`40`	`42`	`/// <inheritdoc/>`
`41`	`43`	`public Task ExecuteAsync(HttpContext httpContext)`
`42`	`44`	`{`