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

Log: Improve feature deprecation/removal logs #3793

Merged
merged 3 commits into from
Sep 12, 2024

Conversation

KobeArthurScofield
Copy link
Contributor

The default logLevel is warning so normally there are not many people know the features that are deprecating.

This pr does these:

  • Changing the feature deprecation warning lo level warning: so normally most people can see the warning, and we can have a good use of it.
  • Add feature removal warning: This indicate the feature has been removed, and can be use as a "REMOVE BEFORE FLIGHT" tag so we can remove this warning after a long enough period of time.

这个改动是像原来设计/现在这样放在 features 里面,还是转移到 errors 比较好?

@yuhan6665 yuhan6665 changed the title Improve feature deprecation/removal logs Log: Improve feature deprecation/removal logs Sep 12, 2024
@yuhan6665 yuhan6665 merged commit 1919a1e into XTLS:main Sep 12, 2024
36 checks passed
@yuhan6665
Copy link
Member

感谢!

@KobeArthurScofield KobeArthurScofield deleted the deprecation-feature-log branch September 12, 2024 16:07
leninalive pushed a commit to amnezia-vpn/amnezia-xray-core that referenced this pull request Oct 29, 2024
* Improve feature deprecation/removal logs

* Add a note about the new feature

* When the feature has removed, this should be an error not a warning
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.

2 participants