K.G. Wang comments

Results 13 comments of


K.G. Wang

报文中掺杂额外字节

另外我怀疑如果只开一个tcp通道，多个udp口同时写会有并发问题（数据错乱），表现是这样：我的应用在KCP之上还带了TLS加密，有时会报Tag错误。不用rsock udp直连不会有问题。开了多个tcp通道（数量远多于udp口时）没有看到这样的异常。

报文中掺杂额外字节

我目前暂时把这样特征的字节段跳过了，并且开了多个tcp通道 https://gist.github.com/wkgcass/9fb70d8c4f5312ef7727ec7542f982fa 暂时是能用了

> > 另外我怀疑如果只开一个tcp通道，多个udp口同时写会有并发问题（数据错乱），表现是这样： > > 我的应用在KCP之上还带了TLS加密，有时会报Tag错误。不用rsock udp直连不会有问题。开了多个tcp通道（数量远多于udp口时）没有看到这样的异常。 > > > f268b10bd0083eed1700b6bc82180c30f129e400000045e318370000000000 > > 你server/client端的mac地址是不是f2:xx? 不是的

Kotlin: Incorrect branch coverage with nested elvis operator

For now I use the following way to bypass the missed branch coverage reports: ```kotlin @lombok.Generated object CoverageUtils { inline fun cast(t: Any?): R { return t as R }...

Ability to modify `proxy.server.list` at runtime

It would be easier and more meaningful to support reloading `proxy.domain.list` first.

Time wheel implementation required

> 接口有点奇怪啊。不应该是类似于这种吗 > > ```java > interface Timer { > void runDelayed(Runnable r, long step, TimeUnit tu); > } > ``` 不应该

Time wheel implementation required

这个接口设计是有原因的。因为selector要根据最近定时任务的时间poll，还要根据是否有定时任务来决定是否永久等待。而且定时任务还可以随时取消。这里除了TimeElem的get方法是个util之外其他都是有实际作用的。

Create proxy_user_build_record.md

server的部署有一键脚本，可以自动生成证书以及启动/重启服务，可以直接用。 [start-vproxy-websocks-proxy.sh](https://github.com/wkgcass/vproxy/blob/dev/misc/start-vproxy-websocks-proxy-server.sh) 拷贝脚本，然后执行即可。客户端的部分可以参考。但是不够明确，比如配置文件修改了哪些地方，根据什么文档修改的。我把这个pr的链接加到README_ZH.md里，有人要看的话可以直接点进来看。你后续如果有时间尽量完善一下，在你的仓库更新时这个PR的预览里也可以同时更新。