michael.heier/jellyfin
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Remove special HTTP HEAD processing

Browse Source 
removing this allows HTTP 206 Partial Content responses and lets some clients(popcorn hour namely) play videos from JF.
This commit is contained in:
SenorSmartyPants 2022-01-07 16:29:22 -06:00
parent 565ebbb643
commit bd2bec4d4a
2 changed files with 0 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Jellyfin.Api/Controllers/ImageController.cs
View File

@ -2020,12 +2020,6 @@ namespace Jellyfin.Api.Controllers
} }
} }
// if the request is a head request, return a NoContent result with the same headers as it would with a GET request
if (isHeadRequest)
{
return NoContent();
}
return PhysicalFile(imagePath, imageContentType ?? MediaTypeNames.Text.Plain); return PhysicalFile(imagePath, imageContentType ?? MediaTypeNames.Text.Plain);
} }
} }

12
Jellyfin.Api/Helpers/FileStreamResponseHelpers.cs
View File

@ -45,12 +45,6 @@ namespace Jellyfin.Api.Helpers
httpContext.Response.Headers[HeaderNames.AcceptRanges] = "none"; httpContext.Response.Headers[HeaderNames.AcceptRanges] = "none";
if (isHeadRequest)
{
httpContext.Response.Headers[HeaderNames.ContentType] = contentType;
return new OkResult();
}
return new FileStreamResult(await response.Content.ReadAsStreamAsync(cancellationToken).ConfigureAwait(false), contentType); return new FileStreamResult(await response.Content.ReadAsStreamAsync(cancellationToken).ConfigureAwait(false), contentType);
} }
@ -70,12 +64,6 @@ namespace Jellyfin.Api.Helpers
{ {
httpContext.Response.ContentType = contentType; httpContext.Response.ContentType = contentType;
// if the request is a head request, return an OkResult (200) with the same headers as it would with a GET request
if (isHeadRequest)
{
return new OkResult();
}
return new PhysicalFileResult(path, contentType) { EnableRangeProcessing = true }; return new PhysicalFileResult(path, contentType) { EnableRangeProcessing = true };
} }