WCF、IErrorHandler 和 log4Net

Question

我正在尝试在 wcf 服务上实现错误处理行为，该服务将使用 log4net 记录异常

[AttributeUsage(AttributeTargets.Class)]
public class AErrorHandlerBehaviorAttribute : Attribute, IServiceBehavior, IErrorHandler{

  private static readonly ILog log = LogManager.GetLogger(System.Reflection.MethodBase.GetCurrentMethod().DeclaringType);

  protected Type ServiceType { get; set; }
  public void Validate(ServiceDescription serviceDescription, ServiceHostBase serviceHostBase)
  {
   //Dont do anything
  }

  public void AddBindingParameters(ServiceDescription serviceDescription, ServiceHostBase serviceHostBase, Collection <ServiceEndpoint> endpoints, BindingParameterCollection bindingParameters)
  {
   //dont do anything
  }

  public void ApplyDispatchBehavior(ServiceDescription serviceDescription, ServiceHostBase serviceHostBase)
  {
   ServiceType = serviceDescription.ServiceType;
   foreach (ChannelDispatcher dispatcher in serviceHostBase.ChannelDispatchers)
   {
    dispatcher.ErrorHandlers.Add(this);
   }
  }

  public void ProvideFault(Exception error, MessageVersion version, ref Message fault)
  {
   fault = null; //Suppress any faults in contract
  }

  public bool HandleError(Exception error)
  {   
   log.Error("Page Load failed : " + error.Message); 
   return false;
  }
}

，然后实现使用该行为的服务。如果我在服务中声明 ILog 变量，则效果很好。

[AErrorHandlerBehavior]
public class AService : IAService
{

     private static readonly ILog log = LogManager.GetLogger(System.Reflection.MethodBase.GetCurrentMethod().DeclaringType);

        ....//various methods
}

但是，当未声明 ILog 变量时，日志记录将停止工作。

[AErrorHandlerBehavior]
public class AService : IAService
{

     //private static readonly ILog log = LogManager.GetLogger(System.Reflection.MethodBase.GetCurrentMethod().DeclaringType);

        ....//various methods
}

理想情况下，我不想在我生成的每个服务中都声明 ILog 变量，特别是当它已经在行为中声明时。

有人可以解释一下 a) 为什么必须在行为和服务中声明这一点。 b) 避免双重声明的任何方法或 c) 登录 wcf 的更好方法。

Answer 1

我不相信您用来解析该 Log 对象的代码：

private static readonly ILog log = 
  LogManager.GetLogger(
    System.Reflection.MethodBase.GetCurrentMethod().DeclaringType
  );

这将始终解析为编写代码行的类型，因为它首先解析自动定义的静态构造函数（使用 MethodInfo.周围类的 GetCurrentMethod()），然后获取其声明类型。

好吧，我说总是...

我不喜欢这种模式，因为它隐式依赖于您尚未亲自编写的代码（因此编译器的一个内部功能 >可能随时更改）；如果您明确地将其放置在您自己编写的静态构造函数中，效果会稍微好一些。

如果你想这样做，那么使用

typeof(_whatever_type_you_declare_it_in_);

...不要依赖编译器和运行时。

同样 - 我有一种感觉，您可能会错误地认为该模式会解析为定义该属性的服务，但实际上不会。

您是否打算能够控制在服务级别使用的日志，并使用行为级别默认值？

如果是这样，那么我建议您：

1) 在您的行为中添加一个名为 _serviceLog 的实例级 ILog 属性

2) 在您的 ApplyDispatchBehaviour 实现中> 这样做：

_serviceLog = LogManager.GetLogger(serviceDescription.ServiceType);

3) 然后你的 HandleError 实现可能如下：

public bool HandleError(Exception error)
{
  //use the service-level log, or a default
  ILog targetLog = _serviceLog ?? log;
  if(targetLog != null)
    targetLog.Error("Page Load failed : " + error.Message);
  return false;
}

Answer 2

实际的问题是，忽略获取记录器的正确方法是什么，在行为中我没有配置记录器。

解决方法只是将配置语句添加到 ApplyDispatchBehavior

public void ApplyDispatchBehavior(ServiceDescription serviceDescription, ServiceHostBase serviceHostBase)
        {
            XmlConfigurator.Configure();            
            foreach (ChannelDispatcher dispatcher in serviceHostBase.ChannelDispatchers)
            {
                dispatcher.ErrorHandlers.Add(this);
            }
        }

我推测我可以在没有此行为的情况下进行日志记录的原因是因为服务中 ILog 的声明导致了错误发生之前记录器的配置抛出