Message and exception provided via LoggingEvent created from LoggingEventData is not sent to Elastic Search

[RedwoodForest]

Message and exception provided via LoggingEvent created from LoggingEventData is not sent to Elastic Search

[urielha]

Hi @lawrencejohnston,
I will look at it ASAP, in the past I removed the reference to loggingEvent.RenderedMessage cause I saw no difference..
Maybe it's my fault.

will update you soon.
Thanks!!

By the way,
Are you sure you want to do this: loggingEvent.Fix = FixFlags.All ?
Consider that this has a performance hit, maybe FixFlags.Partial will be enough for you :)

[RedwoodForest]

Hi @urielha

Thanks!

I'll take a look at FixFlags.All vs FixFlags.Partial for my code. I'm using it to stop log4net from dynamically populating fields that I'm not setting explicitly in the event (like the thread name). It may be that as long as in my appenders I'm only using fields included in FixFlags.Partial that that would be fine.

[urielha]

@lawrencejohnston ,
Are you using ILogger from a reason? why don't you use the regular ILog?

One more thing,
The object you are logging is of Exception type? or it's just the LoggingEvent?
I'm trying to build a test upon it so we can be sure we got the right behavior.

Thanks

[RedwoodForest]

We're using ILogger because it has a Log method that accepts a LoggingEvent (rather than just a string and exception). We need to use a LoggingEvent because the data is coming from a remote system and already has TimeStamp/ThreadName/Exception (serialized to a string) information.

The object we're logging is a LoggingEvent with the ExceptionString property set. Something like this should work for your test:

var eventData = new LoggingEventData {
    ExceptionString = "Any string" // Can make it look more like Exception.ToString() if you want, but any string will do
    // Any other properties you want to test...
};

var loggingEvent = new LoggingEvent(eventData);

ILogger logger = ... // Get ILogger
logger.Log(loggingEvent)

[urielha]

I'm little busy those days so I will look at it ASAP.
Thanks!

[urielha]

Hi @lawrencejohnston,
It's taking me more time than I thought, but I think you can workaround this issue.

Try to insert the ExceptionString into a property of the loggingEvent:

var eventData = new LoggingEventData {
   // ....
};
eventData.Properties["TheException"] = "Exception string";
var loggingEvent = new LoggingEvent(eventData);

ILogger logger = ... // Get ILogger
logger.Log(loggingEvent)

let me know if this worked for you.

And one question:
can you tell me how exactly you extract the ILogger? are you using ILog.Logger member?

Best regards

[RedwoodForest]

Yes, that's how we're getting an ILogger.

That workaround seems like it would work, but for now we're using a local fork with the changes described in the first post. As soon as the issues we've encountered are fixed and released we'll switch back to the NuGet version.

[urielha]

Fixed with commit 560676d