Send milliseconds in timestamp to Logstash. Reduce dependencies. Add TCP (+optional SSL)

marcelog / logger_logstash_backend

Logstash backend for the Elixir Logger

Apache License 2.0

73 stars 40 forks source link

Send milliseconds in timestamp to Logstash. Reduce dependencies. Add TCP (+optional SSL) #22

Closed derekkraan closed 1 year ago

derekkraan commented 6 years ago

I installed this yesterday and noticed that the milliseconds were missing from all my timestamps. This PR fixes that issue. I've also modernized the date / time handling code and made the JSON encoder module configurable (also changed the default to Jason).

I've updated the TravisCI config to remove Elixir 1.3.0 from the build targets.

derekkraan commented 6 years ago

I've since updated my master branch to include support for TCP (with and without TLS) protocol (loosely based on #8)

derekkraan commented 6 years ago

And now also updated to gracefully handle not being able to reach logstash (so your whole app doesn't crash if logstash is down for whatever reason).

derekkraan commented 6 years ago

I've updated this further to add some timeouts. We were having issues where a new deploy of our app wasn't detecting a closed connection for an hour before it crashed and re-opened the connection. That should happen much quicker now.

derekkraan commented 6 years ago

Hi @marcelog, I've just been adding to and improving this PR as I go along, any chance you'll be able to find some time to review it soon?