Skip to content

fix(storage): reduce worker CPU usage - #69

Open
jyxjjj wants to merge 1 commit into
mainfrom
fix-67
Open

jyxjjj wants to merge 1 commit into
mainfrom
fix-67

Conversation

@jyxjjj

@jyxjjj jyxjjj commented Sep 19, 2026

Copy link
Copy Markdown
Member

Summary / 摘要

  • Reduce config persistence CPU usage by deriving one HKDF-backed AES-256-GCM key per load or save operation while retaining an independent random IV for every encrypted field.

  • Add the enc:v2: envelope and preserve enc:v1: PBKDF2 decryption compatibility; legacy values migrate on the next normal save.

  • Await OneDrive token persistence, fold token refreshes into the admin update save, and remove redundant driver initialization.

  • Add regression coverage for low-CPU encryption, independent IVs, v2 persistence, legacy compatibility, and token persistence ordering.

  • This PR has breaking changes.
    / 此 PR 包含破坏性变更。

  • This PR changes public API, config, storage format, or migration behavior.
    / 此 PR 修改了公开 API、配置、存储格式或迁移行为。

  • This PR requires corresponding changes in related repositories.
    / 此 PR 需要关联仓库同步修改。

Related Issues / 关联 Issue

Fixes #67

Testing / 测试

  • Automated tests were not run because this task explicitly prohibited executing repository code, test runners, runtimes, build tools, linters, formatters, and package managers.
  • Static review completed in four rounds; all confirmed findings from the first three rounds were fixed.
  • git diff --check completed successfully.

Checklist / 检查清单

  • I have read CONTRIBUTING.
    / 我已阅读 CONTRIBUTING
  • I confirm this contribution follows the repository license, contribution policy, and code of conduct.
    / 我确认此贡献符合仓库许可证、贡献规范和行为准则。
  • I have formatted the changed code with gofmt, go fmt, or prettier where applicable.
    / 我已按适用情况使用 gofmtgo fmtprettier 格式化变更代码。
  • I have requested review from relevant maintainers or code owners where applicable.
    / 我已在适用情况下请求相关维护者或代码所有者审查。

AI Disclosure / AI 使用声明

  • This PR includes AI-assisted content.
    / 此 PR 包含 AI 辅助内容。

Tools used / 使用工具:

  • ChatGPT
  • Codex
  • GitHub Copilot
  • Claude
  • Gemini
  • Other (please specify) / 其他(请注明):

Usage scope / 使用范围:

  • Code generation / 代码生成

  • Refactoring / 重构

  • Documentation / 文档

  • Tests / 测试

  • Translation / 翻译

  • Review assistance / 审查辅助

  • I have reviewed and validated all AI-assisted content included in this PR.
    / 我已审核并验证此 PR 中的所有 AI 辅助内容。

  • I have ensured that all AI-assisted commits include Co-Authored-By attribution.
    / 我已确保所有 AI 辅助提交都包含 Co-Authored-By 归属信息。

  • I can reproduce all AI-assisted content included in this PR without any AI tools.
    / 我可以在没有任何 AI 工具的情况下重现此 PR 中包含的所有 AI 辅助内容。

- Derive one HKDF-backed AES key per config operation and migrate legacy envelopes on save
- Await OneDrive token persistence and fold admin refresh updates into a single config write
- Remove redundant driver initialization and add config encryption regression coverage

Co-authored-by: Codex <267193182+codex@users.noreply.github.com>
Signed-off-by: jyxjjj <16695261+jyxjjj@users.noreply.github.com>
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
openlist-work edf732f Sep 19 2026, 04:07 PM

@jyxjjj
jyxjjj requested a review from PIKACHUIM September 19, 2026 16:07
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
openlist-tsworkers edf732f Sep 19 2026, 04:08 PM

pikachuren

This comment was marked as outdated.

@jyxjjj

jyxjjj commented Sep 20, 2026

Copy link
Copy Markdown
Member Author

建议直接关吧,Issue也Not Planned吧,这个场景下能优化的只有数据库不要一次更新全部,没法测试具体会不会再超过10ms,收益也不大,可能是Worker本身就做不了这个事。

@PIKACHUIM PIKACHUIM left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AI评审意见有问题请忽略,但需要实际验证一下再合并

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.

[BUG] Cloudflare Workers 免费版极易触发 exceededCpu 10ms 限制

3 participants