mscdex / socksv5

SOCKS protocol version 5 server and client implementations for node.js
MIT License
400 stars 121 forks source link

server crash when user try to connect to socks4 without a way to recover #6

Closed phuongnd08 closed 9 years ago

phuongnd08 commented 9 years ago

I created a socks5 server with socksv5js, and when a client try to connect through different protocol (like socks4), the server crash without a way for me to recover from that error. Please advice.

mscdex commented 9 years ago

This should be fixed in d011967841. Let me know if the master branch works for you too.

phuongnd08 commented 9 years ago

Thanks, that works. Though I would like to add a console.log("error", error_msg) if you don't mind :)