Use buffered readers around our socks connections
While working on #786 (closed), @beth noticed that we do some tiny reads in our SOCKS implementation, and we should probably add a buffered reader to make those not result in tiny syscalls.
This isn't a huge issue; we can do it whenever. The code to change is in arti::socks
and chanmgr::transport::proxied
.