feat: create SOCKS5 StreamDialer #44
Conversation
transport/socks5/packet_dialer.go
Outdated
| "net" | ||
|
|
||
| "github.com/Jigsaw-Code/outline-sdk/transport" | ||
| "golang.org/x/net/proxy" |
There was a problem hiding this comment.
Ugh it seems like the proxy library doesn't currently support UDP, so it may make sense to revert this for now: https://cs.opensource.google/go/x/net/+/master:internal/socks/socks.go;l=233;drc=126a5f3b343c940b1ce677f43b138556311b0999
|
Please take another look. I added unit tests! |
| }() | ||
|
|
||
| // Client | ||
| go func() { |
There was a problem hiding this comment.
nit it seems we don't need put to client logic in the go func, because server is already running concurrently.
There was a problem hiding this comment.
I think it's clearer to have the client and server well delineated.
| }() | ||
|
|
||
| // Client | ||
| go func() { |
There was a problem hiding this comment.
I think it's clearer to have the client and server well delineated.
Needs test