From d80842e5170f19112f33919d5cac9d2bf18dce05 Mon Sep 17 00:00:00 2001 From: kdeng00 Date: Mon, 29 Jul 2019 20:59:22 -0400 Subject: [PATCH] Changed song streaming to use only FileStream instead of FileStream and MemoryStream --- Controllers/v1/SongStreamController.cs | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/Controllers/v1/SongStreamController.cs b/Controllers/v1/SongStreamController.cs index c741f52..372d496 100644 --- a/Controllers/v1/SongStreamController.cs +++ b/Controllers/v1/SongStreamController.cs @@ -47,17 +47,14 @@ namespace Icarus.Controllers.V1 var song = songStore.GetSong(new Song { Id = id }); - var mem = new MemoryStream(); - - using (var stream = new FileStream(song.SongPath, FileMode.Open, FileAccess.Read)) - await stream.CopyToAsync(mem); - - mem.Position = 0; + var stream = new FileStream(song.SongPath, FileMode.Open, FileAccess.Read); + stream.Position = 0; + var filename = $"{song.Title}.mp3"; _logger.LogInformation("Starting to stream song...>"); Console.WriteLine("Starting to streamsong..."); - return File(mem, "application/octet-stream", Path.GetFileName(song.SongPath)); + return File(stream, "application/octet-stream", filename); } #endregion }