logger_logstash_backend icon indicating copy to clipboard operation
logger_logstash_backend copied to clipboard

Published 20 hours ago • verified publisher icon marcelog

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

Open derekkraan opened this issue 7 years ago • 4 comments

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 avatar Apr 19 '18 07:04 derekkraan

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

derekkraan avatar Jun 27 '18 13:06 derekkraan

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 avatar Jul 20 '18 07:07 derekkraan

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 avatar Aug 29 '18 12:08 derekkraan

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?

derekkraan avatar Aug 31 '18 07:08 derekkraan