diff --git a/src/NzbDrone.Common.Test/Http/HttpClientFixture.cs b/src/NzbDrone.Common.Test/Http/HttpClientFixture.cs index 61e71db62..b87e3a4ce 100644 --- a/src/NzbDrone.Common.Test/Http/HttpClientFixture.cs +++ b/src/NzbDrone.Common.Test/Http/HttpClientFixture.cs @@ -322,31 +322,6 @@ public void should_not_download_file_with_error() ExceptionVerification.ExpectedWarns(1); } - [Test] - public void should_not_write_redirect_content_to_stream() - { - var file = GetTempFilePath(); - - using (var fileStream = new FileStream(file, FileMode.Create)) - { - var request = new HttpRequest($"http://{_httpBinHost}/redirect/1"); - request.AllowAutoRedirect = false; - request.ResponseStream = fileStream; - - var response = Subject.Get(request); - - response.StatusCode.Should().Be(HttpStatusCode.Redirect); - } - - ExceptionVerification.ExpectedErrors(1); - - File.Exists(file).Should().BeTrue(); - - var fileInfo = new FileInfo(file); - - fileInfo.Length.Should().Be(0); - } - [Test] public void should_send_cookie() { diff --git a/src/NzbDrone.Common/Http/Dispatchers/ManagedHttpDispatcher.cs b/src/NzbDrone.Common/Http/Dispatchers/ManagedHttpDispatcher.cs index 0e4ef9239..cba300e39 100644 --- a/src/NzbDrone.Common/Http/Dispatchers/ManagedHttpDispatcher.cs +++ b/src/NzbDrone.Common/Http/Dispatchers/ManagedHttpDispatcher.cs @@ -127,10 +127,9 @@ public HttpResponse GetResponse(HttpRequest request, CookieContainer cookies) { try { - if (request.ResponseStream != null && httpWebResponse.StatusCode == HttpStatusCode.OK) + if (request.ResponseStream != null) { // A target ResponseStream was specified, write to that instead. - // But only on the OK status code, since we don't want to write failures and redirects. responseStream.CopyTo(request.ResponseStream); } else