search

tuffk / Logstash-output-to-Azure-Blob

Plugin to send Logstash output to Azure blob
Other
8 stars 15 forks source link

Would be better to set default "codec" #28

Closed zysimplelife closed 5 years ago

zysimplelife commented 5 years ago

I tried to use this plugin with default configuration. It did not work properly, I got the output as follows. After several hours troubleshooting, it turn out that the default codes doesn't work with dynamic fields. 

016-03-28T11:33:23.000Z 10.x.x.10 %{message}
2016-03-28T11:33:23.000Z 10.x.x.10 %{message}
2016-03-28T11:33:23.000Z 10.x.x.10 %{message}
2016-03-28T11:33:23.000Z 10.x.x.10 %{message}
2016-03-28T11:33:23.000Z 10.x.x.10 %{message}

After add the codec in the code, it works as my expectation. I noticed that this plugin comes from logstash-output-s3, but very old version. now in that plugin, the default codec has been added.

It would be good to add that one in the code as follows . default :codec, "line" or json format