loveuall
commented
7 years ago 一. 你引用第三方库解决了 socks5 问题，然后呢？config.json 配置呢？多用户呢？是不是还可以两三行代码解决？ 二. 这个是多端口设置密码的时候，让每个端口跑一个 mux，一般用户配置的端口不会超过10个。直接用一个 goroutine 跑一个 mux， 并且不会超过 10个 goroutine，有什么问题？为何需要 sync.WaitGroup来控制？最后的 一个 mux 自然会阻塞主 goroutine，执行了的 goroutine都不需要再等待他们结束. 能不用则不用。 三. 我的 readme 要修改下，socks5 确实不是我自己写的代码，本来就是一个协议，自己写也不至于写不出来。最关键的是，socks5 本来就是一个超简单版本，远远没有标准的 socks5 那么标准。（可参考shadowsocks的 socks5，也没有用户授权的，但局域网里还授权干嘛？） 但是它代码够简单，这个你没有意见吧？并且够用，你也没有意见吧？它的执行效率肯定有标准的一样甚至更快的速度，也没有意见吧？ 四. 透明代理就更简单了。如果你要引入别人的包，你得引入两个，一个 socks5 一个 透明代理。 五. ipv6 要支持并不难，透明代理与 socks5 我都可以轻松支持，就是我没有测试环境，所以我也没有加。我代码里也许有注释说 ipv6 待支持。未来可以加上。 六. 说实话你的初忠是好，你可以提出更有意见的修改方案，不引入复杂性，效果好，肯定会采取行动。毕竟我也想轻松解决master合并分支的问题。
buaazp
一. readme中提到：
对此我不认同，kcptun源项目客户端使用无需单独配置socks5代理，直接设置kcptun client的监听地址就可以进行代理。我自己开了分之，仅仅在server端里加了几行socks5代理的代码，就可以直接使用了，client无需任何修改：https://github.com/buaazp/kcptun/tree/self-proxy 所以我认为你所谓的优化实际上是负优化：
二. https://github.com/loveuall/kcptun/blob/master/cmd/server/main_server.go#L452 循环起多个goroutine的话，用sync.WaitGroup来控制，你这map range里面加i判断长度的方法，我就不评价了。
三. 你声称自己实现了部分socks5的功能（看注释风格完全不像你自己写的，如果是从别处拷过来开源代码请著名作者和license），说是为了不引入3rd包，代码简单，但是你这些代码功能太少，连基本的socks认证，ipv6都不支持，毕竟你不是专门解析socks5的库，为何不直接调用成熟的第三方socks包，连云风菊苣都表示自己实现这个太复杂而放弃了，看这儿。