fw876 / helloworld

GNU General Public License v3.0
3.64k stars 2.64k forks source link

fix minor errors for tuic-features #1223

Closed AmberisMyShiba closed 1 year ago

AmberisMyShiba commented 1 year ago

fix variables in client-config.lua fix minor error in gen_config.lua

zxlhhyccc commented 1 year ago

使用此pr后gen_config.lua引起ssrp无法运行: image 去掉这些划圈就可正常运行: image

zxlhhyccc commented 1 year ago

实际上,把gen_config.lua中我截图划圈部分删除即可正常运行,不需修改其他部分: image

AmberisMyShiba commented 1 year ago

实际上,把gen_config.lua中我截图划圈部分删除即可正常运行,不需修改其他部分: image

Thanks for indicating the errors, but your commits are incomplete, I think it will make some confusion to the maintainers. I've fixed the issues you guys encountered. Thanks again.

leochien0102 commented 1 year ago

这里只是前面的变量值拼一个s字符,用以显示 ### s(秒)

zxlhhyccc commented 1 year ago

这里只是前面的变量值拼一个s字符,用以显示 ### s(秒)

我理解,但是加了"s"后却不能运行,请看报错提示! image

leochien0102 commented 1 year ago

这里只是前面的变量值拼一个s字符,用以显示 ### s(秒)

我理解,但是加了"s"后却不能运行,请看报错提示! image

这个错误不是因为连接了nil值报的错么,为啥前面的变量值(这个heartbeat)会是nil,找下原因咯

zxlhhyccc commented 1 year ago

这个错误不是因为连接了nil值报的错么,为啥前面的变量值(这个heartbeat)会是nil,找下原因咯

我把client-config.lua、gen_config.lua文件比对了,一样的代码,但是就是报错,我把”s“所有行的变量值都加上”“号即正常,是否可行: image

zxlhhyccc commented 1 year ago

这里只是前面的变量值拼一个s字符,用以显示 ### s(秒)

大佬,大雕合并了我的pr,说明大雕认可我的修改,你这个pr是绝对无法正常运行。。。。不知道你的pr你这边为何能运行!

leochien0102 commented 1 year ago

这里只是前面的变量值拼一个s字符,用以显示 ### s(秒)

大佬,大雕合并了我的pr,说明大雕认可我的修改,你这个pr是绝对无法正常运行。。。。不知道你的pr你这边为何能运行!

你的这个pr,把‘s'去掉只是避免了server.heartbeat值为nil时拼接字符's'而抛出的错误,实际应该解决的是如何让server.heartbeat值不为nil,或者先判断为nil时先行进行处理。至于server.heartbeat这个变量是应该是从tuic客户端配置中读取的

zxlhhyccc commented 1 year ago

这里只是前面的变量值拼一个s字符,用以显示 ### s(秒)

大佬,大雕合并了我的pr,说明大雕认可我的修改,你这个pr是绝对无法正常运行。。。。不知道你的pr你这边为何能运行!

你的这个pr,把‘s'去掉只是避免了server.heartbeat值为nil时拼接字符's'而抛出的错误,实际应该解决的是如何让server.heartbeat值不为nil,或者先判断为nil时先行进行处理。至于server.heartbeat这个变量是应该是从tuic客户端配置中读取的

不仅仅是server.heartbeat变量值读取的是nil,而是server.timeout、server.gc_interval、server.gc_lifetime都读取了nil值。。。

AmberisMyShiba commented 1 year ago

这里只是前面的变量值拼一个s字符,用以显示 ### s(秒)

大佬,大雕合并了我的pr,说明大雕认可我的修改,你这个pr是绝对无法正常运行。。。。不知道你的pr你这边为何能运行!

你的这个pr,把‘s'去掉只是避免了server.heartbeat值为nil时拼接字符's'而抛出的错误,实际应该解决的是如何让server.heartbeat值不为nil,或者先判断为nil时先行进行处理。至于server.heartbeat这个变量是应该是从tuic客户端配置中读取的

不仅仅是server.heartbeat变量值读取的是nil,而是server.timeout、server.gc_interval、server.gc_lifetime都读取了nil值。。。

Cloud you show me what the content of /etc/config/shadowsocksr ? I wander what will happen if you remove the existed tuic node and re-creat a new one. I changed the definition of heartbeat's value type from STRING to INTEGER in my latest commits.

zxlhhyccc commented 1 year ago

此处是否还有一个问题,已控制为tuic,那么native怎么选择的了?: https://github.com/fw876/helloworld/pull/1223/commits/2db06fb263e4582e618c8dcb76bd37c13e1cbb1c#diff-7fac60d9081f69967d8d970bd3eaf93fb33d8f655e22598e7aa70c5d5bd7744cR371-R374

AmberisMyShiba commented 1 year ago

This is my output of server config in /etc/config/shadowsocksr

root@OpenWrt:~# grep heartbeat /etc/config/shadowsocksr
option heartbeat '3'

I guess your output maybe like the below

root@OpenWrt:~# grep heartbeat /etc/config/shadowsocksr
option heartbeat '3s'

I think you just modified a partial content of full source code. There are still some contents you need to change and commit. Good luck!

zxlhhyccc commented 1 year ago

I think you just modified a partial content of full source code. There are still some contents you need to change and commit. Good luck!

我的ssrp启动后/etc/config/shadowsocksr根本没有heartbeat,我认为你是使用tuic。我声明一下: 我没有使用tuic,如果你是使用tuic,你测试一下比如:ss、ssr等不使用tuic的节点,估计就会报我提到的报错问题!

再次对每个文件进行了比对,源代码内容完整。。。

AmberisMyShiba commented 1 year ago

I think you just modified a partial content of full source code. There are still some contents you need to change and commit. Good luck!

我的ssrp启动后/etc/config/shadowsocksr根本没有heartbeat,我认为你是使用tuic。我声明一下: 我没有使用tuic,如果你是使用tuic,你测试一下比如:ss、ssr等不使用tuic的节点,估计就会报我提到的报错问题!

dude.I have to mention that you posted your private information. It's not too late now to hide them. By the way, your PR just maybe solve your issues, but really make the new features broken.

zxlhhyccc commented 1 year ago

dude.I have to mention that you posted your private information. It's not too late now to hide them. By the way, your PR just maybe solve your issues, but really make the new features broken.

谢谢提醒,我把那个私信删除了。起码我的pr可保证除tuic以外的其他节点正常运行(因无tuic的节点故无法测试是否如你一样可以运行),但目前你的代码只解决你使用tuic的问题,对于其他协议的节点,直接报错无法使用,我认为一个源码出来,应该兼容所有节点的协议能正常稳定运行,而不是仅仅解决某个方面的,建议大佬再次优化一下,谢谢!

AmberisMyShiba commented 1 year ago

dude.I have to mention that you posted your private information. It's not too late now to hide them. By the way, your PR just maybe solve your issues, but really make the new features broken.

谢谢提醒,我把那个私信删除了。起码我的pr可保证除tuic以外的其他节点正常运行(因无tuic的节点故无法测试是否如你一样可以运行),但目前你的代码只解决你使用tuic的问题,对于其他协议的节点,直接报错无法使用,我认为一个源码出来,应该兼容所有节点的协议能正常稳定运行,而不是仅仅解决某个方面的,建议大佬再次优化一下,谢谢!