Skip to content

Conversation

Fangliding
Copy link
Member

No description provided.

@Fangliding
Copy link
Member Author

#4944 不是什么重要问题不影响发版

@RPRX
Copy link
Member

RPRX commented Jul 26, 2025

下个版本再合,这种每次发版后再进行旧功能奇葩配置测试然后再给新版报个 panic 的行为属于恶趣味,不能惯着,平时干嘛去了

昨天 v25.7.25 发了两版也是因为 caee152 ,而不是 UDP listen localhost,只是今天又想发版,顺便合了

@RPRX
Copy link
Member

RPRX commented Jul 26, 2025

重要的事情说三遍,如果是针对新功能的测试或引入了新 bug 还能理解,旧功能平时就能测,不要有恶趣味的行为

@xqzr
Copy link
Contributor

xqzr commented Jul 26, 2025

巧合罢了

@RPRX
Copy link
Member

RPRX commented Jul 26, 2025

巧合罢了

还有哪些想测的配置这段时间都测完吧,下个版本前合并修复

@RPRX
Copy link
Member

RPRX commented Jul 26, 2025

不然如果合了这个 PR 重发 v25.7.26,今天或明天又测出个配置 panic,没完没了了

@xqzr
Copy link
Contributor

xqzr commented Jul 26, 2025

今天或明天又测出个配置 panic,没完没了了

panic 删掉就没有 panic 了😋

@RPRX
Copy link
Member

RPRX commented Aug 3, 2025

@xqzr 此时此刻还有别的想测的吗

@RPRX RPRX merged commit d4f11e6 into main Aug 3, 2025
78 checks passed
@RPRX RPRX deleted the UDS branch August 3, 2025 11:02
maoxikun pushed a commit to maoxikun/Xray-core that referenced this pull request Aug 23, 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