lgladsden/jellyfin-server
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Handle IO errors in LoadManifest

Browse Source
This commit is contained in:
Bond_009 2021-01-12 16:03:13 +01:00
parent 1ea2b200c0
commit 1fdd2d6e05
1 changed files with 19 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
Emby.Server.Implementations/Plugins/PluginManager.cs
View File

@ -512,15 +512,20 @@ namespace Emby.Server.Implementations.Plugins
var metafile = Path.Combine(dir, "meta.json");
if (File.Exists(metafile))
{
// Only path where this stays null is when File.ReadAllBytes throws an IOException
byte[] data = null!;
try
{
var data = File.ReadAllBytes(metafile);
data = File.ReadAllBytes(metafile);
manifest = JsonSerializer.Deserialize<PluginManifest>(data, _jsonOptions);
}
catch (IOException ex)
{
_logger.LogError(ex, "Error reading file {Path}.", dir);
}
catch (JsonException ex)
{
_logger.LogError(ex, "Error deserializing {Path}.", dir);
}
_logger.LogError(ex, "Error deserializing {Json}.", Encoding.UTF8.GetString(data!));
}
if (manifest != null)
@ -537,6 +542,7 @@ namespace Emby.Server.Implementations.Plugins
return new LocalPlugin(dir, _appVersion >= targetAbi, manifest);
}
}
// No metafile, so lets see if the folder is versioned.
// TODO: Phase this support out in future versions.