fluent / fluent-logger-forward-node

A fluent forward protocol implementation for Node.js
https://fluent.github.io/fluent-logger-forward-node/
Apache License 2.0
11 stars 8 forks source link

fix reconnect disabled #7

Closed jamiees2 closed 3 years ago

jamiees2 commented 3 years ago

If reconnections are disabled, the FluentClient constructor can throw an unhandledPromiseRejection, which isn't ideal.

Thinking about it, we're not doing a great job of supporting reconnectDisabled, since we should also expose the events on the socket so that the user can decide to reconnect. For now this isn't used, so we can probably come back to it, just like the server :)