Skip to content

Conversation

@xqzr
Copy link
Contributor

@xqzr xqzr commented Jul 25, 2025

Reopen #4927

@Fangliding
Copy link
Member

我那样有问题吗

@xqzr
Copy link
Contributor Author

xqzr commented Jul 25, 2025

我那样有问题吗

没有,但改动比较多。
多一份 PR,多一种选择😉

@RPRX
Copy link
Member

RPRX commented Jul 25, 2025

@Fangliding 你决定用哪个吧

@Fangliding
Copy link
Member

我要自己写肯定是觉得应该合我写的啊
我只是觉得逻辑分散在两个文件不太好 而且全改到dialer和listener可以跨平台

@xqzr xqzr changed the title Inbounds: TCP KeepAlive off By Default & Outbounds: Chrome TCP KeepAlive as Default Value Inbounds & Outbounds: TCP KeepAlive off By Default Jul 25, 2025
@xqzr
Copy link
Contributor Author

xqzr commented Jul 25, 2025

我认为:不应参考 Chrome。
因为,Chrome 空闲连接超时(老化)时间 10 分钟;而 Xray 只有 5 分钟。

@RPRX
Copy link
Member

RPRX commented Jul 25, 2025

不过客户端都 Chrome 指纹了,还是参考一下吧,我觉得要不就简单些先改个默认值,以后再研究 #4931

@RPRX
Copy link
Member

RPRX commented Jul 25, 2025

@xqzr

@Fangliding
Copy link
Member

不过客户端都 Chrome 指纹了,还是参考一下吧,我觉得要不就简单些先改个默认值,以后再研究 #4931

合这个等会另一半都要git冲突麻烦 如果今天就要发版的话这茬玩意推一推也问题不大

@RPRX
Copy link
Member

RPRX commented Jul 25, 2025

#4931 (comment)

@RPRX RPRX closed this Jul 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants