diff --git a/Jellyfin.Server/CoreAppHost.cs b/Jellyfin.Server/CoreAppHost.cs
index 59285a5109..cd5a2ce853 100644
--- a/Jellyfin.Server/CoreAppHost.cs
+++ b/Jellyfin.Server/CoreAppHost.cs
@@ -41,16 +41,6 @@ namespace Jellyfin.Server
networkManager,
configuration)
{
- var previousVersion = ConfigurationManager.CommonConfiguration.PreviousVersion;
- if (ApplicationVersion.CompareTo(previousVersion) > 0)
- {
- Logger.LogWarning("Version check shows Jellyfin was updated: previous version={0}, current version={1}", previousVersion, ApplicationVersion);
-
- // TODO: run update routines
-
- ConfigurationManager.CommonConfiguration.PreviousVersion = ApplicationVersion;
- ConfigurationManager.SaveConfiguration();
- }
}
///
@@ -67,5 +57,33 @@ namespace Jellyfin.Server
///
protected override void ShutdownInternal() => Program.Shutdown();
+
+ ///
+ /// Runs the migration routines if necessary.
+ ///
+ public void TryMigrate()
+ {
+ var previousVersion = ConfigurationManager.CommonConfiguration.PreviousVersion;
+ switch (ApplicationVersion.CompareTo(previousVersion))
+ {
+ case 1:
+ Logger.LogWarning("Version check shows Jellyfin was updated: previous version={0}, current version={1}", previousVersion, ApplicationVersion);
+
+ Migrations.Run(this, Logger);
+
+ ConfigurationManager.CommonConfiguration.PreviousVersion = ApplicationVersion;
+ ConfigurationManager.SaveConfiguration();
+ break;
+ case 0:
+ // nothing to do, versions match
+ break;
+ case -1:
+ Logger.LogWarning("Version check shows Jellyfin was rolled back, use at your own risk: previous version={0}, current version={1}", previousVersion, ApplicationVersion);
+ // no "rollback" routines for now
+ ConfigurationManager.CommonConfiguration.PreviousVersion = ApplicationVersion;
+ ConfigurationManager.SaveConfiguration();
+ break;
+ }
+ }
}
}
diff --git a/Jellyfin.Server/Migrations.cs b/Jellyfin.Server/Migrations.cs
new file mode 100644
index 0000000000..95fea4ea55
--- /dev/null
+++ b/Jellyfin.Server/Migrations.cs
@@ -0,0 +1,92 @@
+using System;
+using System.Collections.Generic;
+using MediaBrowser.Common.Configuration;
+using MediaBrowser.Model.Configuration;
+using Microsoft.Extensions.Logging;
+
+namespace Jellyfin.Server
+{
+ ///
+ /// The class that knows how migrate between different Jellyfin versions.
+ ///
+ internal static class Migrations
+ {
+ private static readonly IUpdater[] _migrations =
+ {
+ new Pre10_5()
+ };
+
+ ///
+ /// Interface that descibes a migration routine.
+ ///
+ private interface IUpdater
+ {
+ ///
+ /// Gets maximum version this Updater applies to.
+ /// If current version is greater or equal to it, skip the updater.
+ ///
+ public abstract Version Maximum { get; }
+
+ ///
+ /// Execute the migration from version "from".
+ ///
+ /// Host that hosts current version.
+ /// Host logger.
+ /// Version to migrate from.
+ /// Whether configuration was changed.
+ public abstract bool Perform(CoreAppHost host, ILogger logger, Version from);
+ }
+
+ ///
+ /// Run all needed migrations.
+ ///
+ /// CoreAppHost that hosts current version.
+ /// AppHost logger.
+ /// Whether anything was changed.
+ public static bool Run(CoreAppHost host, ILogger logger)
+ {
+ bool updated = false;
+ var version = host.ServerConfigurationManager.CommonConfiguration.PreviousVersion;
+
+ for (var i = 0; i < _migrations.Length; i++)
+ {
+ var updater = _migrations[i];
+ if (version.CompareTo(updater.Maximum) >= 0)
+ {
+ logger.LogDebug("Skipping updater {0} as current version {1} >= its maximum applicable version {2}", updater, version, updater.Maximum);
+ continue;
+ }
+
+ if (updater.Perform(host, logger, version))
+ {
+ updated = true;
+ }
+
+ version = updater.Maximum;
+ }
+
+ return updated;
+ }
+
+ private class Pre10_5 : IUpdater
+ {
+ public Version Maximum { get => Version.Parse("10.5.0"); }
+
+ public bool Perform(CoreAppHost host, ILogger logger, Version from)
+ {
+ // Set EnableThrottling to false as it wasn't used before, and in 10.5.0 it may introduce issues
+ var encoding = ((IConfigurationManager)host.ServerConfigurationManager).GetConfiguration("encoding");
+ if (encoding.EnableThrottling)
+ {
+ logger.LogInformation("Disabling transcoding throttling during migration");
+ encoding.EnableThrottling = false;
+
+ host.ServerConfigurationManager.SaveConfiguration("encoding", encoding);
+ return true;
+ }
+
+ return false;
+ }
+ }
+ }
+}
diff --git a/Jellyfin.Server/Program.cs b/Jellyfin.Server/Program.cs
index 484e507a23..aa1bdb1691 100644
--- a/Jellyfin.Server/Program.cs
+++ b/Jellyfin.Server/Program.cs
@@ -182,6 +182,7 @@ namespace Jellyfin.Server
// A bit hacky to re-use service provider since ASP.NET doesn't allow a custom service collection.
appHost.ServiceProvider = host.Services;
appHost.FindParts();
+ appHost.TryMigrate();
try
{