jjchiw / gelf4net

GELF log4net Appender - graylog2
MIT License
63 stars 59 forks source link

AMQP tested / ActivateOptions event to construct correct transport #1

Closed RussPAll closed 12 years ago

RussPAll commented 12 years ago

Hi there,

I've tested the AMQP implementation and (after a tiny bug fix) it's now correctly firing messages into my RabbitMq server.

I've added the ActivateOptions event to the appender so that it can now construct the appropriate Transport class, rather than having to construct both classes and invoke correct one via a switch in the Append method.

I've also done a little work on the unit tests and extracted a GelfJsonBuilder class which if needed could be tested independently.

Thanks,

Russ