Fix graphite client reconnections on Node 16 and above

jonathansamines commented 2 years ago

Fix graphite client reconnections by bundling lazy-socket into this module. Work was divided into multiple commits:

7872a93f68c4f7cc8060ce4fb3c1ef3c1b4af621 - Adds newer Node.js versions to the test matrix
a3c9d476fa6b643bff0ed3babbc1c02e8dcf89a2 - Moves lazy-socket source code and tests into this module. Source code was copied as last published in npm. Tests are failing in Node.js 16 at this point as described by https://github.com/felixge/node-lazy-socket/issues/3
770002151ed7999aee8dec7ff1c202a162dea600 - Modifies lazy-socket source code to restore the automatic reconnection functionality on terminated connections

Serphentas commented 2 years ago

LGTM. I think we should release this as a new patch version (i.e. 0.1.5) since there are no major changes (API is untouched).

Serphentas commented 2 years ago

jonathansamines commented 2 years ago

Hey @Serphentas thanks for reviewing these changes. Have they already been published to npm?

Serphentas commented 2 years ago

I'm having trouble logging into my npm account, I will let you know when it's published ASAP.

Serphentas commented 2 years ago

jonathansamines commented 2 years ago

Awesome, thank you @Serphentas

felixge / node-graphite