From 2392060bc1cc1dc14f83e49c30e918522fe3362a Mon Sep 17 00:00:00 2001
From: Deluan <deluan@deluan.com>
Date: Thu, 2 Apr 2020 22:17:52 -0400
Subject: [PATCH] Don't try to transcode a file if the requested format is the
 same and the client is not requesting to downsample

---
 engine/media_streamer.go      | 4 ++++
 engine/media_streamer_test.go | 7 +++++++
 2 files changed, 11 insertions(+)

diff --git a/engine/media_streamer.go b/engine/media_streamer.go
index dfcf91bf..5cc9bed5 100644
--- a/engine/media_streamer.go
+++ b/engine/media_streamer.go
@@ -147,6 +147,10 @@ func selectTranscodingOptions(ctx context.Context, ds model.DataStore, mf *model
 	if reqFormat == "raw" {
 		return
 	}
+	if reqFormat == mf.Suffix && reqBitRate == 0 {
+		bitRate = mf.BitRate
+		return
+	}
 	trc, hasDefault := ctx.Value("transcoding").(model.Transcoding)
 	var cFormat string
 	var cBitRate int
diff --git a/engine/media_streamer_test.go b/engine/media_streamer_test.go
index 755eea8b..48c30fec 100644
--- a/engine/media_streamer_test.go
+++ b/engine/media_streamer_test.go
@@ -104,6 +104,13 @@ var _ = Describe("MediaStreamer", func() {
 				Expect(format).To(Equal("mp3"))
 				Expect(bitRate).To(Equal(192))
 			})
+			It("returns raw if requested format is the same as the original, but requested BitRate is 0", func() {
+				mf.Suffix = "mp3"
+				mf.BitRate = 320
+				format, bitRate := selectTranscodingOptions(ctx, ds, mf, "mp3", 0)
+				Expect(format).To(Equal("raw"))
+				Expect(bitRate).To(Equal(320))
+			})
 		})
 
 		Context("player has format configured", func() {