Castle Windsor not supplying NLog component when using ASP.NET MVC filters - asp.net-mvc-4

Question

Registration
container.AddFacility<LoggingFacility>(f => f.LogUsing(LoggerImplementation.NLog)
.WithConfig("NLog.config"));
This code is as per this documentation page.
Now this is the code where NLog will be used i.e. Castle Windsor will supply NLog as the _logger implementation:
public class EmailController : Controller
{
private ILogger _logger = NullLogger.Instance;
...
public ILogger Logger
{
get { return _logger; }
set { _logger = value; }
}
public ActionResult Send(UserMessageModel userMessage, bool captchaValid, string captchaErrorMessage)
{
if (ModelState.IsValid)
{
try
{
// Do something
}
catch (Exception ex)
{
_logger.Error(string.Format("Error msg:{0}\rError stacktrace:{1}", ex.Message, ex.StackTrace));
...
}
}
...
}
}
However if I change the code thus:
[CustomHandleError]
public class EmailController : Controller
{
[CustomHandleError]
public ActionResult Send(UserMessageModel userMessage, bool captchaValid, string captchaErrorMessage)
{
if (ModelState.IsValid)
{
// Do something
}
...
}
...
}
public class CustomHandleError: HandleErrorAttribute
{
private ILogger _logger = NullLogger.Instance;
public ILogger Logger
{
get { return _logger; }
set { _logger = value; }
}
public override void OnException(ExceptionContext filterContext)
{
if (filterContext == null)
{
base.OnException(null);
}
_logger.Error(string.Format("Error msg:{0}\rError stacktrace:{1}", filterContext.Exception.Message, filterContext.Exception.StackTrace));
if (filterContext.HttpContext.IsCustomErrorEnabled)
{
filterContext.ExceptionHandled = true;
base.OnException(filterContext);
}
}
}
In this case the _logger is "NullLogger.Instance". I thought that the fact that I had a "Logger" property meant that Castle Windsor would then change _logger to NLog similar to the previous code.
What could I be misunderstanding?
EDIT
I've now added the code here and if anyone can have a look at it that would be great.
FYI: There are 4 commits.
The first commit was the initial commit
The second commit was showing that I could get NLog to work as I required
The third commit was demonstrating that I could get the Data Attribute working i.e. I successfully registered the Filters in Castle Windsor
The last commit is the issue I described above i.e. I can't get the centralised logging to use NLog.

Answer 1

Filters are not created automatically through the container. You need to leverage IActionInvoker into your controller factory.
Read this post
Castle Windsor - Injecting IActionInvoker Implementation Issue
EDIT
Your custom filter needs to derive from ActionFilterAttribute instead of HandleErrorAttribute in order to have dependencies injected by the container (leveraging on the IActionInvoker implemented in your sample).
If the purpose of your filter is handling errors, implements IExceptionFilter as well.