Nice beginning for a socks library!

This a a problem often encountered with asyncdispatch and the Nim std library 
itself, but if you want to use it for production projects, I'd recommend you 
audit the code for leaks and error handling in general - code like 
<https://github.com/enthus1ast/nimSocks/blob/19defc5fe3e131efdd2f4544dbb71323cb1bf853/nimSocks/server.nim#L159>
 for example is not exception safe and is prone to cause the server to leak 
file descriptors - this is tied to the use of `asyncCheck` which in itself is 
broken in this regard and should probably be avoided.

Reply via email to