Merge pull request #436 from cvium/remove_weird_admin_code

Remove RegisterServerWithAdministratorAccess
This commit is contained in:
Joshua M. Boniface 2019-01-05 17:09:48 -05:00 committed by GitHub
commit af6c4279ee

View File

@ -1310,7 +1310,6 @@ namespace Emby.Server.Implementations
{ {
if (!ServerConfigurationManager.Configuration.IsPortAuthorized) if (!ServerConfigurationManager.Configuration.IsPortAuthorized)
{ {
RegisterServerWithAdministratorAccess();
ServerConfigurationManager.Configuration.IsPortAuthorized = true; ServerConfigurationManager.Configuration.IsPortAuthorized = true;
ConfigurationManager.SaveConfiguration(); ConfigurationManager.SaveConfiguration();
} }
@ -2225,32 +2224,6 @@ namespace Emby.Server.Implementations
protected abstract void ShutdownInternal(); protected abstract void ShutdownInternal();
/// <summary>
/// Registers the server with administrator access.
/// </summary>
private void RegisterServerWithAdministratorAccess()
{
Logger.LogInformation("Requesting administrative access to authorize http server");
try
{
AuthorizeServer();
}
catch (NotImplementedException)
{
}
catch (Exception ex)
{
Logger.LogError(ex, "Error authorizing server");
}
}
protected virtual void AuthorizeServer()
{
throw new NotImplementedException();
}
public event EventHandler HasUpdateAvailableChanged; public event EventHandler HasUpdateAvailableChanged;
private bool _hasUpdateAvailable; private bool _hasUpdateAvailable;