Fix Issue #59 Corner case in the retry buffering logic when buffering is set to zero

uber-archive / multitransport-jsonrpc

JSON-RPC Client (Node.js & Browser) and Server (Node.js) aim at "natural looking" server and client code.

116 stars 22 forks source link

Fix Issue #59 Corner case in the retry buffering logic when buffering is set to zero #60

Closed dfellis closed 9 years ago

dfellis commented 9 years ago

A refactor a year ago broke the semantics here.

This was not noticed because any non-trivial usage of the TCP Client should not allow messages to buffer indefinitely.

coveralls commented 9 years ago

Coverage remained the same when pulling 30b52e40ae4495f16a1fcb7752cb369db615cc8b on fix-retry-logic-in-buffering-case into 9acfd493eb6493db4c3dbfc5557abef243a99e73 on master.