Revert "feat: enable upgrade delivery by default"

[zhaohuiw42]

Reverts #366

[github-actions]

CLA Assistant Lite bot:
提交邮箱中包含我们的合作伙伴，但您似乎并非合作伙伴的成员或对接人，请联系相关对接人将您添加至组织之中，或由其重新发起 Pull Request。
The commit email domain belongs to one of our partners, but it seems you are not yet a member of the current organization, please contact the contact person to add you to the organization or let them submit the Pull Request.

_{You can retrigger this bot by commenting recheck in this Pull Request}

[deepin-ci-robot]

deepin pr auto review

这段代码是一个配置文件（JSON格式）的差异对比，具体涉及的是 Deepin Linux 发行版中 lastore（应用商店/包管理服务）的一个配置项。

以下是对该修改的详细审查意见，涵盖逻辑、质量和安全方面：

1. 语法逻辑

• 审查结果：通过。
• 分析：JSON 语法完全正确。修改内容是将布尔值 true 改为 false，结构完整，没有语法错误。

2. 代码质量

• 审查结果：良好，但需注意配置的一致性。
• 分析：
  • 修改明确且单一，仅改变了 upgrade-delivery-enabled 的状态。
  • 建议：请确认 serial 字段（当前为 0）是否需要递增。在某些配置管理系统中（如 dconfig），serial 用于标识配置版本的变更。如果该系统依赖 serial 来判断配置是否需要更新或同步，仅仅修改 value 而不修改 serial 可能会导致配置更新不生效或被覆盖。建议检查该系统的配置规范，如果需要，应将 "serial": 0 修改为 "serial": 1。

3. 代码性能

• 审查结果：无直接影响。
• 分析：这是一个静态配置项的修改。它不会直接影响代码的执行效率或算法复杂度。
• 间接影响：
  • 设为 true 时：系统可能会在后台自动连接服务器检查更新或下载更新包。这会消耗网络带宽和磁盘 I/O，但在用户需要更新时能提供更快的体验。
  • 设为 false 时：系统不会主动进行这些后台操作，节省了后台资源（CPU、网络、I/O），但用户手动检查更新时可能会花费更长时间（因为需要现场下载）。

4. 代码安全

• 审查结果：安全性提升。
• 分析：
  • 关闭自动更新分发 (value: false)：这通常意味着关闭了后台自动下载更新包或分发更新的功能。
  • 优点：
    1. 减少攻击面：关闭后台自动连接更新服务器，减少了潜在的中间人攻击或恶意更新服务器劫持的风险窗口。
    2. 隐私保护：减少了系统向服务器发送请求的频率，有助于保护用户隐私（如 IP 地址、系统版本信息等）。
    3. 防止意外更新：防止了在用户不知情的情况下下载不兼容或有问题的更新包，增强了系统的稳定性。
  • 潜在风险：如果这是完全关闭“更新提醒”或“安全补丁推送”，用户可能会错过重要的安全修复，导致系统长期暴露在已知漏洞下。不过，通常该配置项主要控制“后台静默下载/分发”，而非完全禁止更新，因此整体来看是偏向安全和稳定的选择。

总结与改进建议

该修改将 upgrade-delivery-enabled 设为 false，从逻辑上是合理的，有助于节省系统资源并提升安全性和隐私性。

改进建议：

1. 检查 serial 字段：请确认该配置系统是否要求在 value 变更时同步增加 serial 的值。如果需要，请一并修改。
2. 注释说明：虽然 JSON 标准不支持注释，但在提交代码或文档中，建议注明关闭该功能的具体原因（例如：为了节省流量、为了企业版内网环境部署等），以便后续维护。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: qiuzhiqian, zhaohuiw42

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

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

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