From 31bb5b5cb349f33842994415dd1d876385b599bb Mon Sep 17 00:00:00 2001 From: Luke Pulverenti Date: Wed, 27 Mar 2013 18:13:46 -0400 Subject: [PATCH] removed udp server layer --- .../MediaBrowser.Common.csproj | 2 - MediaBrowser.Common/Net/IUdpServer.cs | 46 ----- .../Net/UdpMessageReceivedEventArgs.cs | 21 --- MediaBrowser.Controller/Kernel.cs | 9 - .../IO/DirectoryWatchers.cs | 4 + ...MediaBrowser.Server.Implementations.csproj | 2 +- .../ServerManager/RegisterServer.bat | 28 --- .../ServerManager/ServerManager.cs | 162 +----------------- .../Udp/UdpServer.cs | 47 +++-- .../ApplicationHost.cs | 90 +++++++++- .../MainWindow.xaml | 8 +- .../MediaBrowser.ServerApplication.csproj | 1 + 12 files changed, 138 insertions(+), 282 deletions(-) delete mode 100644 MediaBrowser.Common/Net/IUdpServer.cs delete mode 100644 MediaBrowser.Common/Net/UdpMessageReceivedEventArgs.cs delete mode 100644 MediaBrowser.Server.Implementations/ServerManager/RegisterServer.bat diff --git a/MediaBrowser.Common/MediaBrowser.Common.csproj b/MediaBrowser.Common/MediaBrowser.Common.csproj index bdef529a57..a8fc4564c7 100644 --- a/MediaBrowser.Common/MediaBrowser.Common.csproj +++ b/MediaBrowser.Common/MediaBrowser.Common.csproj @@ -80,12 +80,10 @@ - - diff --git a/MediaBrowser.Common/Net/IUdpServer.cs b/MediaBrowser.Common/Net/IUdpServer.cs deleted file mode 100644 index 036977eab5..0000000000 --- a/MediaBrowser.Common/Net/IUdpServer.cs +++ /dev/null @@ -1,46 +0,0 @@ -using System; -using System.Threading.Tasks; - -namespace MediaBrowser.Common.Net -{ - /// - /// Interface IUdpServer - /// - public interface IUdpServer : IDisposable - { - /// - /// Occurs when [message received]. - /// - event EventHandler MessageReceived; - - /// - /// Starts the specified port. - /// - /// The port. - void Start(int port); - - /// - /// Stops this instance. - /// - void Stop(); - - /// - /// Sends the async. - /// - /// The bytes. - /// The remote end point. - /// Task. - /// data - Task SendAsync(byte[] bytes, string remoteEndPoint); - - /// - /// Sends the async. - /// - /// The bytes. - /// The ip address. - /// The port. - /// Task. - /// bytes - Task SendAsync(byte[] bytes, string ipAddress, int port); - } -} diff --git a/MediaBrowser.Common/Net/UdpMessageReceivedEventArgs.cs b/MediaBrowser.Common/Net/UdpMessageReceivedEventArgs.cs deleted file mode 100644 index bd5034c47d..0000000000 --- a/MediaBrowser.Common/Net/UdpMessageReceivedEventArgs.cs +++ /dev/null @@ -1,21 +0,0 @@ -using System; - -namespace MediaBrowser.Common.Net -{ - /// - /// Class UdpMessageReceivedEventArgs - /// - public class UdpMessageReceivedEventArgs : EventArgs - { - /// - /// Gets or sets the bytes. - /// - /// The bytes. - public byte[] Bytes { get; set; } - /// - /// Gets or sets the remote end point. - /// - /// The remote end point. - public string RemoteEndPoint { get; set; } - } -} diff --git a/MediaBrowser.Controller/Kernel.cs b/MediaBrowser.Controller/Kernel.cs index d43c505a5e..51d347cafc 100644 --- a/MediaBrowser.Controller/Kernel.cs +++ b/MediaBrowser.Controller/Kernel.cs @@ -124,15 +124,6 @@ namespace MediaBrowser.Controller /// The user data repository. public IUserDataRepository UserDataRepository { get; set; } - /// - /// Gets the UDP server port number. - /// - /// The UDP server port number. - public int UdpServerPortNumber - { - get { return 7359; } - } - private readonly IServerConfigurationManager _configurationManager; /// diff --git a/MediaBrowser.Server.Implementations/IO/DirectoryWatchers.cs b/MediaBrowser.Server.Implementations/IO/DirectoryWatchers.cs index 14d413f1a4..c33975a649 100644 --- a/MediaBrowser.Server.Implementations/IO/DirectoryWatchers.cs +++ b/MediaBrowser.Server.Implementations/IO/DirectoryWatchers.cs @@ -454,6 +454,10 @@ namespace MediaBrowser.Server.Implementations.IO // Should we remove it from it's parent? Logger.ErrorException("Error refreshing {0}", ex, i.Name); } + catch (Exception ex) + { + Logger.ErrorException("Error refreshing {0}", ex, i.Name); + } }))).ConfigureAwait(false); } diff --git a/MediaBrowser.Server.Implementations/MediaBrowser.Server.Implementations.csproj b/MediaBrowser.Server.Implementations/MediaBrowser.Server.Implementations.csproj index 0a20370515..9aac84f29f 100644 --- a/MediaBrowser.Server.Implementations/MediaBrowser.Server.Implementations.csproj +++ b/MediaBrowser.Server.Implementations/MediaBrowser.Server.Implementations.csproj @@ -167,6 +167,7 @@ + @@ -244,7 +245,6 @@ - diff --git a/MediaBrowser.Server.Implementations/ServerManager/RegisterServer.bat b/MediaBrowser.Server.Implementations/ServerManager/RegisterServer.bat deleted file mode 100644 index d762dfaf76..0000000000 --- a/MediaBrowser.Server.Implementations/ServerManager/RegisterServer.bat +++ /dev/null @@ -1,28 +0,0 @@ -rem %1 = http server port -rem %2 = http server url -rem %3 = udp server port -rem %4 = tcp server port (web socket) - -if [%1]==[] GOTO DONE - -netsh advfirewall firewall delete rule name="Port %1" protocol=TCP localport=%1 -netsh advfirewall firewall add rule name="Port %1" dir=in action=allow protocol=TCP localport=%1 - -if [%2]==[] GOTO DONE - -netsh http del urlacl url="%2" user="NT AUTHORITY\Authenticated Users" -netsh http add urlacl url="%2" user="NT AUTHORITY\Authenticated Users" - -if [%3]==[] GOTO DONE - -netsh advfirewall firewall delete rule name="Port %3" protocol=UDP localport=%3 -netsh advfirewall firewall add rule name="Port %3" dir=in action=allow protocol=UDP localport=%3 - -if [%4]==[] GOTO DONE - -netsh advfirewall firewall delete rule name="Port %4" protocol=TCP localport=%4 -netsh advfirewall firewall add rule name="Port %4" dir=in action=allow protocol=TCP localport=%4 - - -:DONE -Exit \ No newline at end of file diff --git a/MediaBrowser.Server.Implementations/ServerManager/ServerManager.cs b/MediaBrowser.Server.Implementations/ServerManager/ServerManager.cs index c1453cd8ba..991ac2e3b5 100644 --- a/MediaBrowser.Server.Implementations/ServerManager/ServerManager.cs +++ b/MediaBrowser.Server.Implementations/ServerManager/ServerManager.cs @@ -7,13 +7,8 @@ using MediaBrowser.Model.Net; using MediaBrowser.Model.Serialization; using System; using System.Collections.Generic; -using System.Diagnostics; -using System.IO; using System.Linq; using System.Net; -using System.Net.Sockets; -using System.Reflection; -using System.Text; using System.Threading; using System.Threading.Tasks; @@ -22,14 +17,8 @@ namespace MediaBrowser.Server.Implementations.ServerManager /// /// Manages the Http Server, Udp Server and WebSocket connections /// - public class ServerManager : IServerManager, IDisposable + public class ServerManager : IServerManager { - /// - /// This is the udp server used for server discovery by clients - /// - /// The UDP server. - private IUdpServer UdpServer { get; set; } - /// /// Both the Ui and server will have a built-in HttpServer. /// People will inevitably want remote control apps so it's needed in the Ui too. @@ -65,11 +54,6 @@ namespace MediaBrowser.Server.Implementations.ServerManager /// private readonly ILogger _logger; - /// - /// The _network manager - /// - private readonly INetworkManager _networkManager; - /// /// The _application host /// @@ -106,26 +90,22 @@ namespace MediaBrowser.Server.Implementations.ServerManager private readonly List _webSocketListeners = new List(); private readonly Kernel _kernel; - + /// /// Initializes a new instance of the class. /// /// The application host. - /// The network manager. /// The json serializer. /// The logger. /// The configuration manager. + /// The kernel. /// applicationHost - public ServerManager(IApplicationHost applicationHost, INetworkManager networkManager, IJsonSerializer jsonSerializer, ILogger logger, IServerConfigurationManager configurationManager, Kernel kernel) + public ServerManager(IApplicationHost applicationHost, IJsonSerializer jsonSerializer, ILogger logger, IServerConfigurationManager configurationManager, Kernel kernel) { if (applicationHost == null) { throw new ArgumentNullException("applicationHost"); } - if (networkManager == null) - { - throw new ArgumentNullException("networkManager"); - } if (jsonSerializer == null) { throw new ArgumentNullException("jsonSerializer"); @@ -138,7 +118,6 @@ namespace MediaBrowser.Server.Implementations.ServerManager _logger = logger; _jsonSerializer = jsonSerializer; _applicationHost = applicationHost; - _networkManager = networkManager; ConfigurationManager = configurationManager; _kernel = kernel; } @@ -148,13 +127,6 @@ namespace MediaBrowser.Server.Implementations.ServerManager /// public void Start() { - if (_applicationHost.IsFirstRun) - { - RegisterServerWithAdministratorAccess(); - } - - ReloadUdpServer(); - ReloadHttpServer(); if (!SupportsNativeWebSocket) @@ -162,7 +134,7 @@ namespace MediaBrowser.Server.Implementations.ServerManager ReloadExternalWebSocketServer(); } - ConfigurationManager.ConfigurationUpdated += _kernel_ConfigurationUpdated; + ConfigurationManager.ConfigurationUpdated += ConfigurationUpdated; } /// @@ -181,8 +153,7 @@ namespace MediaBrowser.Server.Implementations.ServerManager /// /// Restarts the Http Server, or starts it if not currently running /// - /// if set to true [register server on failure]. - private void ReloadHttpServer(bool registerServerOnFailure = true) + private void ReloadHttpServer() { // Only reload if the port has changed, so that we don't disconnect any active users if (HttpServer != null && HttpServer.UrlPrefix.Equals(_kernel.HttpServerUrlPrefix, StringComparison.OrdinalIgnoreCase)) @@ -204,16 +175,6 @@ namespace MediaBrowser.Server.Implementations.ServerManager { _logger.ErrorException("Error starting Http Server", ex); - if (registerServerOnFailure) - { - RegisterServerWithAdministratorAccess(); - - // Don't get stuck in a loop - ReloadHttpServer(false); - - return; - } - throw; } @@ -253,60 +214,6 @@ namespace MediaBrowser.Server.Implementations.ServerManager await Task.WhenAll(tasks).ConfigureAwait(false); } - /// - /// Starts or re-starts the udp server - /// - private void ReloadUdpServer() - { - // For now, there's no reason to keep reloading this over and over - if (UdpServer != null) - { - return; - } - - DisposeUdpServer(); - - try - { - // The port number can't be in configuration because we don't want it to ever change - UdpServer = _applicationHost.Resolve(); - - _logger.Info("Starting udp server"); - - UdpServer.Start(_kernel.UdpServerPortNumber); - } - catch (SocketException ex) - { - _logger.ErrorException("Failed to start UDP Server", ex); - return; - } - - UdpServer.MessageReceived += UdpServer_MessageReceived; - } - - /// - /// Handles the MessageReceived event of the UdpServer control. - /// - /// The source of the event. - /// The instance containing the event data. - async void UdpServer_MessageReceived(object sender, UdpMessageReceivedEventArgs e) - { - var context = "Server"; - - var expectedMessage = String.Format("who is MediaBrowser{0}?", context); - var expectedMessageBytes = Encoding.UTF8.GetBytes(expectedMessage); - - if (expectedMessageBytes.SequenceEqual(e.Bytes)) - { - _logger.Info("Received UDP server request from " + e.RemoteEndPoint); - - // Send a response back with our ip address and port - var response = String.Format("MediaBrowser{0}|{1}:{2}", context, _networkManager.GetLocalIpAddress(), ConfigurationManager.Configuration.HttpServerPortNumber); - - await UdpServer.SendAsync(Encoding.UTF8.GetBytes(response), e.RemoteEndPoint); - } - } - /// /// Sends a message to all clients currently connected via a web socket /// @@ -387,20 +294,6 @@ namespace MediaBrowser.Server.Implementations.ServerManager } } - /// - /// Disposes the udp server - /// - private void DisposeUdpServer() - { - if (UdpServer != null) - { - _logger.Info("Disposing UdpServer"); - - UdpServer.MessageReceived -= UdpServer_MessageReceived; - UdpServer.Dispose(); - } - } - /// /// Disposes the current HttpServer /// @@ -428,46 +321,6 @@ namespace MediaBrowser.Server.Implementations.ServerManager DisposeExternalWebSocketServer(); } - /// - /// Registers the server with administrator access. - /// - private void RegisterServerWithAdministratorAccess() - { - _logger.Info("Requesting administrative access to authorize http server"); - - // Create a temp file path to extract the bat file to - var tmpFile = Path.Combine(ConfigurationManager.CommonApplicationPaths.TempDirectory, Guid.NewGuid() + ".bat"); - - // Extract the bat file - using (var stream = Assembly.GetExecutingAssembly().GetManifestResourceStream("MediaBrowser.Server.Implementations.ServerManager.RegisterServer.bat")) - { - using (var fileStream = File.Create(tmpFile)) - { - stream.CopyTo(fileStream); - } - } - - var startInfo = new ProcessStartInfo - { - FileName = tmpFile, - - Arguments = string.Format("{0} {1} {2} {3}", ConfigurationManager.Configuration.HttpServerPortNumber, - _kernel.HttpServerUrlPrefix, - _kernel.UdpServerPortNumber, - ConfigurationManager.Configuration.LegacyWebSocketPortNumber), - - CreateNoWindow = true, - WindowStyle = ProcessWindowStyle.Hidden, - Verb = "runas", - ErrorDialog = false - }; - - using (var process = Process.Start(startInfo)) - { - process.WaitForExit(); - } - } - /// /// Performs application-defined tasks associated with freeing, releasing, or resetting unmanaged resources. /// @@ -485,7 +338,6 @@ namespace MediaBrowser.Server.Implementations.ServerManager { if (dispose) { - DisposeUdpServer(); DisposeHttpServer(); } } @@ -508,7 +360,7 @@ namespace MediaBrowser.Server.Implementations.ServerManager /// The source of the event. /// The instance containing the event data. /// - void _kernel_ConfigurationUpdated(object sender, EventArgs e) + void ConfigurationUpdated(object sender, EventArgs e) { HttpServer.EnableHttpRequestLogging = ConfigurationManager.Configuration.EnableHttpLevelLogging; diff --git a/MediaBrowser.Server.Implementations/Udp/UdpServer.cs b/MediaBrowser.Server.Implementations/Udp/UdpServer.cs index 375d9274ef..718a25a0d6 100644 --- a/MediaBrowser.Server.Implementations/Udp/UdpServer.cs +++ b/MediaBrowser.Server.Implementations/Udp/UdpServer.cs @@ -1,5 +1,7 @@ -using MediaBrowser.Common.Implementations.NetworkManagement; +using System.Linq; +using MediaBrowser.Common.Implementations.NetworkManagement; using MediaBrowser.Common.Net; +using MediaBrowser.Controller.Configuration; using MediaBrowser.Model.Logging; using System; using System.Net; @@ -13,36 +15,49 @@ namespace MediaBrowser.Server.Implementations.Udp /// /// Provides a Udp Server /// - public class UdpServer : IUdpServer + public class UdpServer : IDisposable { /// - /// Occurs when [message received]. + /// The _logger /// - public event EventHandler MessageReceived; + private readonly ILogger _logger; - /// - /// Gets or sets the logger. - /// - /// The logger. - private ILogger Logger { get; set; } + private readonly INetworkManager _networkManager; + private readonly IServerConfigurationManager _serverConfigurationManager; + /// /// Initializes a new instance of the class. /// /// The logger. - public UdpServer(ILogger logger) + /// The network manager. + public UdpServer(ILogger logger, INetworkManager networkManager, IServerConfigurationManager serverConfigurationManager) { - Logger = logger; + _logger = logger; + _networkManager = networkManager; + _serverConfigurationManager = serverConfigurationManager; } /// /// Raises the event. /// /// The instance containing the event data. - protected virtual void OnMessageReceived(UdpMessageReceivedEventArgs e) + private async void OnMessageReceived(UdpMessageReceivedEventArgs e) { - EventHandler handler = MessageReceived; - if (handler != null) handler(this, e); + var context = "Server"; + + var expectedMessage = String.Format("who is MediaBrowser{0}?", context); + var expectedMessageBytes = Encoding.UTF8.GetBytes(expectedMessage); + + if (expectedMessageBytes.SequenceEqual(e.Bytes)) + { + _logger.Info("Received UDP server request from " + e.RemoteEndPoint); + + // Send a response back with our ip address and port + var response = String.Format("MediaBrowser{0}|{1}:{2}", context, _networkManager.GetLocalIpAddress(), _serverConfigurationManager.Configuration.HttpServerPortNumber); + + await SendAsync(Encoding.UTF8.GetBytes(response), e.RemoteEndPoint); + } } /// @@ -82,7 +97,7 @@ namespace MediaBrowser.Server.Implementations.Udp } catch (Exception ex) { - Logger.ErrorException("Error receiving udp message", ex); + _logger.ErrorException("Error receiving udp message", ex); return Task.FromResult(new UdpReceiveResult(new byte[] { }, new IPEndPoint(IPAddress.Any, 0))); } }) @@ -201,7 +216,7 @@ namespace MediaBrowser.Server.Implementations.Udp await _udpClient.SendAsync(bytes, bytes.Length, new NetworkManager().Parse(remoteEndPoint)).ConfigureAwait(false); - Logger.Info("Udp message sent to {0}", remoteEndPoint); + _logger.Info("Udp message sent to {0}", remoteEndPoint); } } diff --git a/MediaBrowser.ServerApplication/ApplicationHost.cs b/MediaBrowser.ServerApplication/ApplicationHost.cs index e7162e3dd4..b259388335 100644 --- a/MediaBrowser.ServerApplication/ApplicationHost.cs +++ b/MediaBrowser.ServerApplication/ApplicationHost.cs @@ -1,4 +1,8 @@ -using MediaBrowser.Api; +using System.Diagnostics; +using System.Net.Cache; +using System.Net.Http; +using System.Net.Sockets; +using MediaBrowser.Api; using MediaBrowser.Common; using MediaBrowser.Common.Configuration; using MediaBrowser.Common.Constants; @@ -55,6 +59,8 @@ namespace MediaBrowser.ServerApplication /// public class ApplicationHost : BaseApplicationHost, IServerApplicationHost { + private const int UdpServerPort = 7359; + /// /// Gets the server kernel. /// @@ -129,6 +135,12 @@ namespace MediaBrowser.ServerApplication /// The HTTP server. private IHttpServer HttpServer { get; set; } + /// + /// Gets or sets the UDP server. + /// + /// The UDP server. + private UdpServer UdpServer { get; set; } + /// /// The full path to our startmenu shortcut /// @@ -172,7 +184,6 @@ namespace MediaBrowser.ServerApplication RegisterSingleInstance(ServerConfigurationManager); RegisterSingleInstance(() => new AlchemyServer(Logger)); - RegisterSingleInstance(new UdpServer(Logger), false); RegisterSingleInstance(new PismoIsoManager(Logger)); RegisterSingleInstance(new BdInfoExaminer()); @@ -183,7 +194,7 @@ namespace MediaBrowser.ServerApplication HttpServer = ServerFactory.CreateServer(this, ProtobufSerializer, Logger, "Media Browser", "index.html"); RegisterSingleInstance(HttpServer, false); - ServerManager = new ServerManager(this, NetworkManager, JsonSerializer, Logger, ServerConfigurationManager, ServerKernel); + ServerManager = new ServerManager(this, JsonSerializer, Logger, ServerConfigurationManager, ServerKernel); RegisterSingleInstance(ServerManager); UserManager = new UserManager(ServerKernel, Logger, ServerConfigurationManager); @@ -247,6 +258,11 @@ namespace MediaBrowser.ServerApplication { base.FindParts(); + //if (IsFirstRun) + { + RegisterServerWithAdministratorAccess(); + } + Parallel.Invoke( () => @@ -261,6 +277,17 @@ namespace MediaBrowser.ServerApplication () => ProviderManager.AddMetadataProviders(GetExports().OrderBy(e => e.Priority).ToArray()) ); + + UdpServer = new UdpServer(Logger, NetworkManager, ServerConfigurationManager); + + try + { + UdpServer.Start(UdpServerPort); + } + catch (SocketException ex) + { + Logger.ErrorException("Failed to start UDP Server", ex); + } } /// @@ -280,6 +307,23 @@ namespace MediaBrowser.ServerApplication get { return ConfigurationManager.CommonConfiguration.EnableAutoUpdate; } } + /// + /// Releases unmanaged and - optionally - managed resources. + /// + /// true to release both managed and unmanaged resources; false to release only unmanaged resources. + protected override void Dispose(bool dispose) + { + if (dispose) + { + if (UdpServer != null) + { + UdpServer.Dispose(); + } + } + + base.Dispose(dispose); + } + /// /// Checks for update. /// @@ -363,6 +407,46 @@ namespace MediaBrowser.ServerApplication public override void Shutdown() { App.Instance.Dispatcher.Invoke(App.Instance.Shutdown); + } + + /// + /// Registers the server with administrator access. + /// + private void RegisterServerWithAdministratorAccess() + { + Logger.Info("Requesting administrative access to authorize http server"); + + // Create a temp file path to extract the bat file to + var tmpFile = Path.Combine(ConfigurationManager.CommonApplicationPaths.TempDirectory, Guid.NewGuid() + ".bat"); + + // Extract the bat file + using (var stream = Assembly.GetExecutingAssembly().GetManifestResourceStream("MediaBrowser.ServerApplication.RegisterServer.bat")) + { + using (var fileStream = File.Create(tmpFile)) + { + stream.CopyTo(fileStream); + } + } + + var startInfo = new ProcessStartInfo + { + FileName = tmpFile, + + Arguments = string.Format("{0} {1} {2} {3}", ServerConfigurationManager.Configuration.HttpServerPortNumber, + ServerKernel.HttpServerUrlPrefix, + UdpServerPort, + ServerConfigurationManager.Configuration.LegacyWebSocketPortNumber), + + CreateNoWindow = true, + WindowStyle = ProcessWindowStyle.Hidden, + Verb = "runas", + ErrorDialog = false + }; + + using (var process = Process.Start(startInfo)) + { + process.WaitForExit(); + } } } } diff --git a/MediaBrowser.ServerApplication/MainWindow.xaml b/MediaBrowser.ServerApplication/MainWindow.xaml index 330bacd355..bbc7e64c8f 100644 --- a/MediaBrowser.ServerApplication/MainWindow.xaml +++ b/MediaBrowser.ServerApplication/MainWindow.xaml @@ -2,7 +2,13 @@ xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:tb="http://www.hardcodet.net/taskbar" - Title="Media Browser Server" Height="350" Width="525" AllowsTransparency="True" Background="Transparent" WindowStyle="None" ShowInTaskbar="False"> + Title="Media Browser Server" + Height="350" + Width="525" + AllowsTransparency="True" + Background="Transparent" + WindowStyle="None" + ShowInTaskbar="False"> diff --git a/MediaBrowser.ServerApplication/MediaBrowser.ServerApplication.csproj b/MediaBrowser.ServerApplication/MediaBrowser.ServerApplication.csproj index ddb55ac6f9..cc71d5df66 100644 --- a/MediaBrowser.ServerApplication/MediaBrowser.ServerApplication.csproj +++ b/MediaBrowser.ServerApplication/MediaBrowser.ServerApplication.csproj @@ -258,6 +258,7 @@ Settings.Designer.cs +