Simplify middleware workflow.

Author: JimBobSquarePants

src/ImageSharp.Web.Providers.Azure/Providers/AzureBlobStorageImageProvider.cs

@@ -8,7 +8,6 @@
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Http.Extensions;
 using Microsoft.Extensions.Options;
-using Microsoft.IO;
 using SixLabors.ImageSharp.Web.Resolvers;
 using SixLabors.ImageSharp.Web.Resolvers.Azure;
 
@@ -35,12 +34,6 @@ private readonly Dictionary<string, BlobContainerClient> containers
         /// </summary>
         private readonly AzureBlobStorageImageProviderOptions storageOptions;
 
-        /// <summary>
-        /// The recyclable memorystream manager used for managing pooled stream
-        /// buffers independently from image buffer pooling.
-        /// </summary>
-        private readonly RecyclableMemoryStreamManager memoryStreamManager;
-
         /// <summary>
         /// Contains various helper methods based on the current configuration.
         /// </summary>
@@ -55,21 +48,14 @@ private readonly Dictionary<string, BlobContainerClient> containers
         /// Initializes a new instance of the <see cref="AzureBlobStorageImageProvider"/> class.
         /// </summary>
         /// <param name="storageOptions">The blob storage options.</param>
-        /// <param name="memoryStreamManager">
-        /// The recyclable memorystream manager used for managing pooled stream
-        /// buffers independently from image buffer pooling.
-        /// </param>
         /// <param name="formatUtilities">Contains various format helper methods based on the current configuration.</param>
         public AzureBlobStorageImageProvider(
             IOptions<AzureBlobStorageImageProviderOptions> storageOptions,
-            RecyclableMemoryStreamManager memoryStreamManager,
             FormatUtilities formatUtilities)
         {
             Guard.NotNull(storageOptions, nameof(storageOptions));
-            Guard.NotNull(memoryStreamManager, nameof(memoryStreamManager));
 
             this.storageOptions = storageOptions.Value;
-            this.memoryStreamManager = memoryStreamManager;
             this.formatUtilities = formatUtilities;
 
             foreach (AzureBlobContainerClientOptions container in this.storageOptions.BlobContainers)
@@ -136,7 +122,7 @@ public async Task<IImageResolver> GetAsync(HttpContext context)
                 return null;
             }
 
-            return new AzureBlobStorageImageResolver(blob, this.memoryStreamManager);
+            return new AzureBlobStorageImageResolver(blob);
         }
 
         /// <inheritdoc/>

src/ImageSharp.Web.Providers.Azure/Resolvers/AzureBlobStorageImageResolver.cs

@@ -7,7 +7,6 @@
 using Azure;
 using Azure.Storage.Blobs;
 using Azure.Storage.Blobs.Models;
-using Microsoft.IO;
 
 namespace SixLabors.ImageSharp.Web.Resolvers.Azure
 {
@@ -16,22 +15,13 @@ namespace SixLabors.ImageSharp.Web.Resolvers.Azure
     /// </summary>
     public class AzureBlobStorageImageResolver : IImageResolver
     {
-        private readonly RecyclableMemoryStreamManager memoryStreamManager;
         private readonly BlobClient blob;
 
         /// <summary>
         /// Initializes a new instance of the <see cref="AzureBlobStorageImageResolver"/> class.
         /// </summary>
         /// <param name="blob">The Azure blob.</param>
-        /// <param name="memoryStreamManager">
-        /// The recyclable memorystream manager used for managing pooled stream
-        /// buffers independently from image buffer pooling.
-        /// </param>
-        public AzureBlobStorageImageResolver(BlobClient blob, RecyclableMemoryStreamManager memoryStreamManager)
-        {
-            this.blob = blob;
-            this.memoryStreamManager = memoryStreamManager;
-        }
+        public AzureBlobStorageImageResolver(BlobClient blob) => this.blob = blob;
 
         /// <inheritdoc/>
         public async Task<ImageMetadata> GetMetaDataAsync()
@@ -42,16 +32,6 @@ public async Task<ImageMetadata> GetMetaDataAsync()
 
         /// <inheritdoc/>
         public async Task<Stream> OpenReadAsync()
-        {
-            // Copy to a MemoryStream first because RetriableStreamImpl
-            // doesn't support Position.
-            Stream blobStream = (await this.blob.DownloadAsync()).Value.Content;
-            var memoryStream = new RecyclableMemoryStream(this.memoryStreamManager);
-
-            await blobStream.CopyToAsync(memoryStream);
-            memoryStream.Seek(0, SeekOrigin.Begin);
-
-            return memoryStream;
-        }
+            => (await this.blob.DownloadAsync()).Value.Content;
     }
 }