Goroutine leak

[vboginskey]

The anonymous goroutine introduced in 4ca7e7b will never exit unless the command is aborted. This leaks a goroutine per successful invocation of ScanStream.

[Freeaqingme]

Good catch! I just looked at the code in which I used this library and I think I simply always called close on the abort channel.

Clearly this was way before context.Context became more commonly used, so my suggestion would be to rewrite it a little to use a context instead.

For what it's worth, I did use it in production extensively. So it is possible to use without goroutine leaks.