Skip to content

fix: update default logging rules in dtk preference#107

Merged
18202781743 merged 1 commit intolinuxdeepin:masterfrom
18202781743:master
Jun 27, 2025
Merged

fix: update default logging rules in dtk preference#107
18202781743 merged 1 commit intolinuxdeepin:masterfrom
18202781743:master

Conversation

@18202781743
Copy link
Contributor

Changed the default logging rules value from empty string to
"*.debug=false" in org.deepin.dtk.preference.json
This modification ensures debug logs are disabled by default for DTK
applications
The change helps reduce unnecessary debug output in production
environments while maintaining the ability to enable debug logging when
needed

fix: 更新 dtk 首选项中的默认日志规则

将 org.deepin.dtk.preference.json 中的默认日志规则值从空字符串改为
"*.debug=false"
此修改确保 DTK 应用程序默认禁用调试日志
该更改有助于减少生产环境中不必要的调试输出,同时保留需要时启用调试日志的
能力

Changed the default logging rules value from empty string to
"*.debug=false" in org.deepin.dtk.preference.json
This modification ensures debug logs are disabled by default for DTK
applications
The change helps reduce unnecessary debug output in production
environments while maintaining the ability to enable debug logging when
needed

fix: 更新 dtk 首选项中的默认日志规则

将 org.deepin.dtk.preference.json 中的默认日志规则值从空字符串改为
"*.debug=false"
此修改确保 DTK 应用程序默认禁用调试日志
该更改有助于减少生产环境中不必要的调试输出,同时保留需要时启用调试日志的
能力
@18202781743
Copy link
Contributor Author

@deepin-ci-robot
Copy link
Contributor

deepin pr auto review

关键摘要:

  • 修改了日志规则配置,添加了 "*.debug=false" 以禁用调试日志。

是否建议立即修改:

  • 否,这个修改看起来是有意为之的,旨在提高应用程序的日志记录性能和安全性。但是,需要确保这个修改不会影响应用程序的正常运行和调试需求。如果这个修改是经过充分测试的,并且不会引入新的问题,那么可以保留这个修改。

@18202781743
Copy link
Contributor Author

linuxdeepin/dtkcore#486

@18202781743 18202781743 requested a review from zccrs June 26, 2025 12:37
@deepin-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, mhduiy

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@18202781743 18202781743 merged commit ea1d174 into linuxdeepin:master Jun 27, 2025
15 of 17 checks passed
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