Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove legacy dns code #3821

Closed

Conversation

KobeArthurScofield
Copy link
Contributor

移除 app/dns 中目前已经被标明的过时功能代码。
没有新的功能停用或弃用。

ProtoBreak:该 pr 合并后,使用 pb 配置启动 core 需要重新生成 pb。

删史一时爽,陈年老 test 火葬场

@Fangliding
Copy link
Member

我记得以前好像在哪说过proto不该break所以前人才只标注弃用没删的
当然这些基本不影响使用的东西怎么搞到底也只是些哲学观点

@KobeArthurScofield
Copy link
Contributor Author

proto 的话一般在同一个版本内使用最好,跨版本兼容某种程度只能看承诺了,而且已经彻底移除了的功能 proto 里面还存在的话到了实际用 pb 配置的时候实在不好发现功能已经被清了
清理的话能尽量干净点最好吧,弃用 proto 多了不止发胖,还容易引来考古学家

@yuhan6665
Copy link
Member

proto 的话一般在同一个版本内使用最好,跨版本兼容某种程度只能看承诺了,而且已经彻底移除了的功能 proto 里面还存在的话到了实际用 pb 配置的时候实在不好发现功能已经被清了 清理的话能尽量干净点最好吧,弃用 proto 多了不止发胖,还容易引来考古学家

不用的删就删吧 但是 proto 的序号应该保持 比如
repeated NameServer name_server = 5; 中的 5
#3817 这个里面同理

@KobeArthurScofield
Copy link
Contributor Author

不用的删就删吧 但是 proto 的序号应该保持

ok 没问题
只 break 这次还是回滚序号?

@KobeArthurScofield
Copy link
Contributor Author

所有 proto 序号已经完成回滚

@RPRX
Copy link
Member

RPRX commented Sep 17, 2024

目的重复,请把这个 pr 的内容合并至 #3817

@RPRX RPRX closed this Sep 17, 2024
KobeArthurScofield added a commit to KobeArthurScofield/Xray-core that referenced this pull request Sep 17, 2024
@KobeArthurScofield
Copy link
Contributor Author

已完成迁移

@KobeArthurScofield KobeArthurScofield deleted the remove-legacy-dns-code branch September 18, 2024 07:56
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.

4 participants