Skip to content

fix(deploy): ecosystem 透传 SSO_JWT_SECRET(修 PM2 启动崩溃)+ MyTask index env#22

Open
jhfnetboy wants to merge 1 commit into
masterfrom
fix/deploy-env-sso-mytask
Open

fix(deploy): ecosystem 透传 SSO_JWT_SECRET(修 PM2 启动崩溃)+ MyTask index env#22
jhfnetboy wants to merge 1 commit into
masterfrom
fix/deploy-env-sso-mytask

Conversation

@jhfnetboy

Copy link
Copy Markdown
Member

问题

PR#18 的 SsoService 对缺失 SSO_JWT_SECRET fail-fast(启动即崩),但 ecosystem.config.js(PM2 生产部署配置)未透传该 env#18 合并后,Docker/PM2 部署的后端会启动崩溃

修复

  • ecosystem.config.js 后端 env 块透传 SSO_JWT_SECRET(须 ≠ JWT_SECRET)+ SSO_ALLOWED_REDIRECTS(空=fail-closed)+ MyTask indexer 的 MYTASK_ESCROW_ADDRESS/MYTASK_INDEX_FROM_BLOCK(后者有默认,空则跳过不崩)
  • .env.example 补 MyTask index 两条(SSO 两条 feat(sso): MV-1 MyVote SSO 通道(cos72 半)— 3 轮安全审闭环 #18 已有)
  • node -e require 校验 ecosystem 语法通过

纯部署配置一致性,无代码逻辑。

https://claude.ai/code/session_01MCxVCnKkpzi5sCGf1FZ4CH

…Task index env

PR#18 让 SsoService 对缺失 SSO_JWT_SECRET fail-fast,但 PM2 生产配置未透传该 env
→ Docker/PM2 部署的后端合并后会启动崩溃。补 SSO_JWT_SECRET/SSO_ALLOWED_REDIRECTS +
MyTask indexer 的 MYTASK_ESCROW_ADDRESS/MYTASK_INDEX_FROM_BLOCK(后者有默认不阻塞);
.env.example 补 MyTask index 两条。

Claude-Session: https://claude.ai/code/session_01MCxVCnKkpzi5sCGf1FZ4CH
@jhfnetboy
jhfnetboy requested a review from fanhousanbu as a code owner July 15, 2026 03:59

@clestons clestons left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

APPROVE — 干净的部署配置修复,无密钥泄露。

ecosystem.config.jsSSO_JWT_SECRET: process.env.SSO_JWT_SECRET 引用环境变量,绝不硬编码进提交文件 —— 正确。这补上了 #18 SsoService boot 时对 SSO_JWT_SECRET 的 fail-fast(否则 PM2/Docker 后端启动即崩)。SSO_ALLOWED_REDIRECTS 默认 ""(fail-closed 全拒),.env.example 里只有公开合约地址 + FROM_BLOCK,无秘密。同时透传 #20 indexer 的 MYTASK_ESCROW_ADDRESS(空则跳过不崩)。无阻塞。

— clestons (Opus review)

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.

2 participants