Skip to content

Config: Remove legacy config fields again#3782

Merged
yuhan6665 merged 6 commits intoXTLS:mainfrom
KobeArthurScofield:rm-legacy-2
Sep 11, 2024
Merged

Config: Remove legacy config fields again#3782
yuhan6665 merged 6 commits intoXTLS:mainfrom
KobeArthurScofield:rm-legacy-2

Conversation

@KobeArthurScofield
Copy link
Contributor

Luckily there seems no more hidden undocumented configs that were deprecated and should be removed now.

收回 #3776 里面这句话

现在新发现里面还埋藏的远古配置代码:

  • routing 下面的 settings :里面的配置早已被移动到 routing 下面,而且已经标记为弃用已久。
  • 配置文件根里的 port :该来源应该为古老 V2Ray 版本中还没分化出 inbound 和 outbound 时的配置项,早已弃用。

迟早还能发现有什么东西 deprecated 但是还留在那里的,明年能清理完吗不知道

@Fangliding
Copy link
Member

Test boomed

@KobeArthurScofield
Copy link
Contributor Author

Test boomed

深埋已久的测试炸弹已拆除 已解决

@KobeArthurScofield
Copy link
Contributor Author

再过了一次应该没有什么新发现了

@yuhan6665 yuhan6665 merged commit 6b1bf31 into XTLS:main Sep 11, 2024
@yuhan6665
Copy link
Member

感谢!

@KobeArthurScofield KobeArthurScofield deleted the rm-legacy-2 branch September 12, 2024 06:01
leninalive pushed a commit to amnezia-vpn/amnezia-xray-core that referenced this pull request Oct 29, 2024
* Remove more lecacy fields

* Patch missing bracket

* Fix tests

* Fix missing comma

* Fix buried test bomb

* Cleanup test after removed legacy test content
it2konst pushed a commit to it2konst/gametunnel-core that referenced this pull request Mar 1, 2026
* Remove more lecacy fields

* Patch missing bracket

* Fix tests

* Fix missing comma

* Fix buried test bomb

* Cleanup test after removed legacy test content
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