search

fluent / fluent-logger-node

A structured logger for Fluentd (Node.js)
Apache License 2.0
259 stars 83 forks source link

Fix packets on multiple tags get corrput and multiple calls of callbacks on error #155

Closed mitsos1os closed 4 years ago

mitsos1os commented 4 years ago

Fix issues #153 #154

This pull request fixes the following:

  1. The size given to write engine now is correct for currently processed tag and packets are correct.
  2. Emit callbacks now are either called with error or without when ack required. (Instead of calling them two times)
  3. Replaced unnecessary usage of let for variable declaration with const
okkez commented 4 years ago

Great work! Thanks!