From fdafa596832eae13cebcf5bbe5fa867f7ba068f0 Mon Sep 17 00:00:00 2001 From: LukePulverenti Date: Thu, 21 Feb 2013 20:26:35 -0500 Subject: Removed System.Windows.Forms dependancy from Common. Almost done removing NLog dependancy. --- MediaBrowser.Common/Net/AlchemyWebSocket.cs | 1 - MediaBrowser.Common/Net/BaseRestService.cs | 12 +++++------ MediaBrowser.Common/Net/Handlers/BaseHandler.cs | 27 +++++-------------------- MediaBrowser.Common/Net/HttpServer.cs | 4 ++-- MediaBrowser.Common/Net/WebSocketConnection.cs | 3 +-- 5 files changed, 14 insertions(+), 33 deletions(-) (limited to 'MediaBrowser.Common/Net') diff --git a/MediaBrowser.Common/Net/AlchemyWebSocket.cs b/MediaBrowser.Common/Net/AlchemyWebSocket.cs index 5a5103f74..584efa999 100644 --- a/MediaBrowser.Common/Net/AlchemyWebSocket.cs +++ b/MediaBrowser.Common/Net/AlchemyWebSocket.cs @@ -1,5 +1,4 @@ using Alchemy.Classes; -using MediaBrowser.Common.Logging; using MediaBrowser.Common.Serialization; using MediaBrowser.Model.Logging; using System; diff --git a/MediaBrowser.Common/Net/BaseRestService.cs b/MediaBrowser.Common/Net/BaseRestService.cs index cfec9398d..59818a932 100644 --- a/MediaBrowser.Common/Net/BaseRestService.cs +++ b/MediaBrowser.Common/Net/BaseRestService.cs @@ -1,7 +1,6 @@ using MediaBrowser.Common.Extensions; using MediaBrowser.Common.IO; using MediaBrowser.Common.Kernel; -using MediaBrowser.Common.Logging; using MediaBrowser.Model.Logging; using ServiceStack.Common; using ServiceStack.Common.Web; @@ -22,17 +21,18 @@ namespace MediaBrowser.Common.Net /// public class BaseRestService : Service, IRestfulService { - /// - /// The logger - /// - protected ILogger Logger = LogManager.GetLogger("RestService"); - /// /// Gets or sets the kernel. /// /// The kernel. public IKernel Kernel { get; set; } + /// + /// Gets or sets the logger. + /// + /// The logger. + public ILogger Logger { get; set; } + /// /// Gets a value indicating whether this instance is range request. /// diff --git a/MediaBrowser.Common/Net/Handlers/BaseHandler.cs b/MediaBrowser.Common/Net/Handlers/BaseHandler.cs index 95c86e6f7..3b3630047 100644 --- a/MediaBrowser.Common/Net/Handlers/BaseHandler.cs +++ b/MediaBrowser.Common/Net/Handlers/BaseHandler.cs @@ -1,7 +1,5 @@ using MediaBrowser.Common.Extensions; using MediaBrowser.Common.Kernel; -using MediaBrowser.Common.Logging; -using MediaBrowser.Model.Logging; using System; using System.Collections; using System.Collections.Generic; @@ -23,12 +21,6 @@ namespace MediaBrowser.Common.Net.Handlers public abstract class BaseHandler : IHttpServerHandler where TKernelType : IKernel { - /// - /// Gets the logger. - /// - /// The logger. - protected ILogger Logger { get; private set; } - /// /// Initializes the specified kernel. /// @@ -36,7 +28,6 @@ namespace MediaBrowser.Common.Net.Handlers public void Initialize(IKernel kernel) { Kernel = (TKernelType)kernel; - Logger = SharedLogger.Logger; } /// @@ -303,7 +294,7 @@ namespace MediaBrowser.Common.Net.Handlers response.ContentType = "text/plain"; - Logger.ErrorException("Error processing request", ex); + //Logger.ErrorException("Error processing request", ex); if (!string.IsNullOrEmpty(ex.Message)) { @@ -321,7 +312,7 @@ namespace MediaBrowser.Common.Net.Handlers } catch (Exception ex1) { - Logger.ErrorException("Error dumping stack trace", ex1); + //Logger.ErrorException("Error dumping stack trace", ex1); } } @@ -449,7 +440,7 @@ namespace MediaBrowser.Common.Net.Handlers if (Kernel.Configuration.EnableHttpLevelLogging) { - Logger.LogMultiline(msg, LogSeverity.Debug, log); + //Logger.LogMultiline(msg, LogSeverity.Debug, log); } } @@ -604,7 +595,7 @@ namespace MediaBrowser.Common.Net.Handlers } catch (Exception ex) { - Logger.ErrorException("Error disposing compressed stream", ex); + //Logger.ErrorException("Error disposing compressed stream", ex); } } @@ -615,7 +606,7 @@ namespace MediaBrowser.Common.Net.Handlers } catch (Exception ex) { - Logger.ErrorException("Error disposing response", ex); + //Logger.ErrorException("Error disposing response", ex); } } @@ -750,14 +741,6 @@ namespace MediaBrowser.Common.Net.Handlers } } - internal static class SharedLogger - { - /// - /// The logger - /// - internal static ILogger Logger = LogManager.GetLogger("Http Handler"); - } - /// /// Class ResponseInfo /// diff --git a/MediaBrowser.Common/Net/HttpServer.cs b/MediaBrowser.Common/Net/HttpServer.cs index 7183b63c0..184b82c16 100644 --- a/MediaBrowser.Common/Net/HttpServer.cs +++ b/MediaBrowser.Common/Net/HttpServer.cs @@ -4,7 +4,6 @@ using MediaBrowser.Common.Kernel; using MediaBrowser.Model.Logging; using ServiceStack.Api.Swagger; using ServiceStack.Common.Web; -using ServiceStack.Logging; using ServiceStack.Logging.NLogger; using ServiceStack.ServiceHost; using ServiceStack.ServiceInterface.Cors; @@ -144,6 +143,7 @@ namespace MediaBrowser.Common.Net } container.Register(Kernel); + container.Register(_logger); foreach (var service in Kernel.RestServices) { @@ -155,7 +155,7 @@ namespace MediaBrowser.Common.Net Serialization.JsonSerializer.Configure(); - LogManager.LogFactory = new NLogFactory(); + ServiceStack.Logging.LogManager.LogFactory = new NLogFactory(); } /// diff --git a/MediaBrowser.Common/Net/WebSocketConnection.cs b/MediaBrowser.Common/Net/WebSocketConnection.cs index 24b8e2a9f..d274d390d 100644 --- a/MediaBrowser.Common/Net/WebSocketConnection.cs +++ b/MediaBrowser.Common/Net/WebSocketConnection.cs @@ -1,5 +1,4 @@ -using MediaBrowser.Common.Logging; -using MediaBrowser.Common.Serialization; +using MediaBrowser.Common.Serialization; using MediaBrowser.Model.Logging; using System; using System.Net; -- cgit v1.2.3