feat: add private permission support for dconfig - #1184
Conversation
The following config items are accessed by applications other than the owner appId, so their visibility should be public instead of private: - org.deepin.dde.daemon.account: isAllowLocalUnlockTerminal (accessed by dde-session-shell greeter) - org.deepin.dde.daemon.timedate: timeZone (accessed by dde-control-center datetime module) - org.deepin.dde.daemon.keyboard: capslockToggle (accessed by dde-shell key-notify applet) - org.deepin.Display: defaultTemperatureManual (accessed by dde-control-center display module) Serial bumped from 0 to 1 for each changed item. PMS: BUG-000000
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 18202781743 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdds private permission configuration for dconfig-related daemons by updating several DSG config JSONs to support per-app private access control, aligning them with the new private-permission scheme described in the PR description. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
deepin pr auto review★ 总体评分:55分■ 【总体评价】
■ 【详细分析】
■ 【改进建议代码示例】 --- a/misc/dsg-configs/org.deepin.dde.daemon.account.json
+++ b/misc/dsg-configs/org.deepin.dde.daemon.account.json
@@ -14,13 +14,13 @@
},
"isAllowLocalUnlockTerminal": {
"value": false,
- "serial": 0,
+ "serial": 1,
"flags": ["global"],
"name": "isAllowLocalUnlockTerminal",
"name[zh_CN]": "是否容许本地解锁终端",
"description": "Whether to allow local unlocking of the terminal",
- "permissions": "readwrite",
- "visibility": "private"
+ "permissions": "readonly",
+ "visibility": "public"
},
"passwordEncryptionAlgorithm": {
"value": "sm3", |
|
@18202781743: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
feat: add private permission management for dconfig daemon
变更说明
具体修改的配置项
技术方案简述
关联的其他 PR
Summary by Sourcery
New Features: