pt
/
jellyfin
mirror of https://github.com/jellyfin/jellyfin.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

total font size limit 20M

This commit is contained in:
Nyanmisaka 2020-07-30 17:36:38 +08:00 committed by GitHub
parent 893bc7400b
commit 8165c3bb21
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
MediaBrowser.Api/Subtitles/SubtitleService.cs
View File

@ -382,8 +382,6 @@ namespace MediaBrowser.Api.Subtitles
{
try
{
// max single font size 10M
var maxSize = 10485760;
var fontFile = _fileSystem.GetFiles(fallbackFontPath)
.First(i => string.Equals(i.Name, request.Name, StringComparison.OrdinalIgnoreCase));
var fileSize = fontFile?.Length;
@ -391,13 +389,7 @@ namespace MediaBrowser.Api.Subtitles
if (fileSize != null && fileSize > 0)
{
Logger.LogDebug("Fallback font size is {0} Bytes", fileSize);
if (fileSize <= maxSize)
{
return await ResultFactory.GetStaticFileResult(Request, fontFile.FullName);
}
Logger.LogWarning("The selected font is too large. Maximum allowed size is 10 Megabytes");
return await ResultFactory.GetStaticFileResult(Request, fontFile.FullName);
}
else
{