Skip to content

为远程发布补充文件密钥支持#38

Open
walker1211 wants to merge 1 commit into
caol64:mainfrom
walker1211:feat/publish-api-key-file
Open

为远程发布补充文件密钥支持#38
walker1211 wants to merge 1 commit into
caol64:mainfrom
walker1211:feat/publish-api-key-file

Conversation

@walker1211

Copy link
Copy Markdown
Contributor

背景

PR #36wenyan serve 增加了 --api-key-file,解决了服务端启动参数暴露 API Key 的问题,但当时没有覆盖 wenyan publish --server 的远程客户端场景。客户端仍只能使用 --api-key,真实密钥会出现在进程命令行中。

本 PR 补齐这个遗漏,使远程发布客户端也能从权限受限的文件读取密钥。

修改

  • wenyan publish 增加 --api-key-file <path>
  • 复用 servepublish 的文件读取、首尾空白清理及错误处理逻辑。
  • 禁止同时使用 --api-key--api-key-file
  • 将文件内容读入内存后,只把 API Key 交给现有远程发布逻辑,不继续传递文件路径。
  • 更新 README、发布文档和 Docker 客户端示例,默认推荐文件密钥。

兼容性

  • 保留 --api-key,已有调用不受影响。
  • --api-key-file 仅在指定 --server 的远程发布模式下生效。
  • 本 PR 不直接修改版本号;如需发布,适合作为下一个 patch 版本 2.0.11

验证

  • pnpm run test(42 项通过)
  • pnpm run typecheck
  • pnpm run build
  • pnpm run lint(无新增错误)
  • wenyan publish --help 显示 --api-key-file

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.

1 participant