Ability to add additional fields via log4j

[nstielau]

Hey,

I'm looking for a way to specify some additional fields via the log4j.properties. In my specific use-case, I'd like to add an environment (dev/staging/production) field, as well as an application (ThisService,ThatService) field. Wondering if you had any thoughts on a general solution. Maybe there could be an 'additionalFields' property whose setter could take query-string like values ('environment=dev&application=myApp'). I could modify the appender for my needs, but this feels like a pretty common use-case. Let me know if you have any ideas. Thanks,

-nick

[t0xa]

Hi Nick,

I thought if GELF is using JSON already why not to pass additional fields as JSON string. Please check my latest commit and tell me if it's okay to use it like this.

Another way is to detect some sort of additional property file (e.g. "gelf.properties") and add fields from it but I'm not sure if this is a good practice.

Anton

[nstielau]

AWESOME! That's amazing. Very flexible and clean solution. Thanks for the quick turn-around. I'll try it out soon. -nick