Changed to IFileInfo instead of file name string

Author: khellang

src/Microsoft.AspNet.Http.Abstractions/HttpResponse.cs

@@ -6,6 +6,7 @@
 using System.Threading;
 using System.Threading.Tasks;
 using Microsoft.AspNet.Http.Features;
+using Microsoft.Extensions.FileProviders;
 
 namespace Microsoft.AspNet.Http
 {
@@ -109,41 +110,50 @@ public abstract class HttpResponse
         /// <summary>
         /// Sends the given file using the SendFile extension.
         /// </summary>
-        /// <param name="fileName">The full path to the file.</param>
+        /// <param name="file">The file to send.</param>
         /// <returns></returns>
-        public virtual Task SendFileAsync(string fileName)
+        public virtual Task SendFileAsync(IFileInfo file)
         {
-            if (fileName == null)
+            if (file == null)
             {
-                throw new ArgumentNullException(nameof(fileName));
+                throw new ArgumentNullException(nameof(file));
             }
 
-            return SendFileAsync(fileName, 0, null, CancellationToken.None);
+            return SendFileAsync(file, 0, null, CancellationToken.None);
         }
 
         /// <summary>
         /// Sends the given file using the SendFile extension.
         /// </summary>
-        /// <param name="fileName">The full path to the file.</param>
+        /// <param name="file">The file to send.</param>
         /// <param name="offset">The offset in the file.</param>
         /// <param name="count">The number of types to send, or null to send the remainder of the file.</param>
         /// <param name="cancellationToken"></param>
         /// <returns></returns>
-        public virtual Task SendFileAsync(string fileName, long offset, long? count, CancellationToken cancellationToken = default(CancellationToken))
+        public virtual Task SendFileAsync(IFileInfo file, long offset, long? count, CancellationToken cancellationToken = default(CancellationToken))
         {
-            if (fileName == null)
+            if (file == null)
             {
-                throw new ArgumentNullException(nameof(fileName));
+                throw new ArgumentNullException(nameof(file));
+            }
+
+            if (string.IsNullOrEmpty(file.PhysicalPath))
+            {
+                var readStream = file.CreateReadStream();
+
+                readStream.Seek(offset, SeekOrigin.Begin); // TODO: What if !CanSeek?
+
+                return StreamCopyOperation.CopyToAsync(readStream, Body, count, cancellationToken);
             }
 
             var sendFile = HttpContext.Features.Get<IHttpSendFileFeature>();
 
             if (sendFile == null)
             {
-                return SendFileAsync(Body, fileName, offset, count, cancellationToken);
+                return SendFileAsync(Body, file.PhysicalPath, offset, count, cancellationToken);
             }
 
-            return sendFile.SendFileAsync(fileName, offset, count, cancellationToken);
+            return sendFile.SendFileAsync(file.PhysicalPath, offset, count, cancellationToken);
         }
 
         // Not safe for overlapped writes.

src/Microsoft.AspNet.Http.Abstractions/project.json

@@ -11,6 +11,7 @@
   },
   "dependencies": {
     "Microsoft.AspNet.Http.Features": "1.0.0-*",
+    "Microsoft.Extensions.FileProviders.Abstractions": "1.0.0-*",
     "Microsoft.Extensions.ActivatorUtilities.Sources": {
       "type": "build",
       "version": "1.0.0-*"

src/Microsoft.AspNet.Http.Extensions/project.json

@@ -16,6 +16,6 @@
   },
   "frameworks": {
     "net451": {},
-    "dotnet5.4": { }
+    "dotnet5.4": {}
   }
 }

test/Microsoft.AspNet.Http.Extensions.Tests/SendFileResponseExtensionsTests.cs renamed to test/Microsoft.AspNet.Http.Abstractions.Tests/SendFileTests.cs

@@ -8,9 +8,9 @@
 using Microsoft.AspNet.Http.Internal;
 using Xunit;
 
-namespace Microsoft.AspNet.Http.Extensions.Tests
+namespace Microsoft.AspNet.Http.Abstractions.Tests
 {
-    public class SendFileResponseExtensionsTests
+    public class SendFileTests
     {
         [Fact]
         public Task SendFileWhenFileNotFoundThrows()