luci-app-ssr-plus: udp Only used to customize the underlying protoc…
#1381
Conversation
…ol used for QUIC connections
传输 (Transport)
transport 用于自定义 QUIC 连接使用的底层协议。目前唯一可用的类型是 udp，保留类型选项是为了将来可能添加的其他类型。
transport:
type: udp
udp:
hopInterval: 30s
…ol used for QUIC connections
传输 (Transport)
transport 用于自定义 QUIC 连接使用的底层协议。目前唯一可用的类型是 udp，保留类型选项是为了将来可能添加的其他类型。
````
transport:
type: udp
udp:
hopInterval: 30s
````
QUIC 参数 ````` quic: initStreamReceiveWindow: 8388608 maxStreamReceiveWindow: 8388608 initConnReceiveWindow: 20971520 maxConnReceiveWindow: 20971520 maxIdleTimeout: 30s keepAlivePeriod: 10s disablePathMTUDiscovery: false ``````
|
你这么改，genconfig那里还是会出错，正常的流程是：对于在luci页面中填写的配置参数，如果有填，就生成相应配置，如果没有，则用nil或者默认参数。我认为，只要修改/etc/config/shadowsocksr 的option内容。或者完全删除hysteria的节点配置后建立一个全新的节点就可以排除hysteria无法启动的错误。 所以你想提交PR,应该尽量保留前作者们贡献的代码，试图理解前作者的思路，而不是粗暴地删除和修改。如果贡献者有明显的bug，可以指出错误并通知他们更正，如果他们没有时间和精力，你可以修复bug，并为这个库贡献更多的代码。 结论：review后不建议合并这个PR.。建议先确定问题，再修改代码。 |
我这个pr是按照官方的配置规范处理的（见上面的官方配置要求截图，我就担心有人提出疑问才把官方配置要求截图），不属于粗暴的修改原作者的代码（难道原来的代码就不能修改？比如：原作者提交的代码都无法运行也是不能修改吗？），反而是遵循官方配置，烦请大佬先看清楚官方的配置规范要求后再下结论！谢谢！还有就是：经此pr修改，你说genconfig那里还是会出错，请问大佬你测试了吗？！反而经此pr修改后测试发现，软件运行一切正常，而且不启用端口跳跃和UDP协议运行更稳定，断网频率大为减少！ |
|
测试了一下，原来gen_config已经被修改过了，所以，相对于这个PR,不影响gen_config的生成结果，可以正常运行。但是这个PR相当于回撤了原作者的提交，丢弃了一些实用的配置，删除了为了未来升级而保留的配置。所以仓库的维护者请自行定夺是否合并。 另外我想提醒一下@zxlhhyccc，hysteria的问题并非transport中的udp和hopinterval引起，你在debug的时候，可能以为不同的问题，殊途同归，然而我却认为没有找到真正的问题，哪怕毫厘之差，最终谬以千里。 BTW,与你沟通感觉非常suck，尽管你总把谢谢挂在嘴边。 |
原来gen_config本来就有问题导致无法运行所以修改了。。。。，另外，这个pr实际功能仍然保留，只是启用相关功能即可。 |
我上面说的话是有点不好听，关键是你看一下你前面说的什么话咯！对于这个pr，自有仓库的维护者判断，你为何没测试就那么早下结论？！ |
…ol used for QUIC connections
根据官方的配置说明：
udp仅在配置端口跳跃时启用，因此，未配置端口跳跃时，不需默认启用udp协议。见下面的官方部分说明！传输 (Transport)
transport 用于自定义 QUIC 连接使用的底层协议。目前唯一可用的类型是 udp，保留类型选项是为了将来可能添加的其他类型。