fix: 修复http 无法登录问题；优化部分文案

[ding113]

No description provided.

[claude]

潜在问题：降级逻辑可能导致安全风险

当代理失败后自动降级到直连时，移除了 dispatcher 但没有重新创建 init 对象，这可能导致一些与代理相关的配置残留。

建议修改：

// 移除 dispatcher，重新发起请求
const fallbackInit = { ...init };
delete fallbackInit.dispatcher;
try {
  response = await fetch(proxyUrl, fallbackInit);
  logger.info("ProxyForwarder: Direct connection succeeded after proxy failure", {
    providerId: provider.id,
    providerName: provider.name,
  });
} catch (directError) {
  // ...
}

这样可以确保降级请求是完全干净的，避免代理配置残留。

[claude]

改进建议：统一布尔值转换逻辑

当前每个布尔值字段都重复相同的 transform 逻辑，建议提取为公共函数：

// 在文件顶部添加
const booleanTransform = (s: string) => s !== "false" && s !== "0";

// 然后在 schema 中使用
AUTO_MIGRATE: z.string().default("true").transform(booleanTransform),
ENABLE_RATE_LIMIT: z.string().default("true").transform(booleanTransform),
ENABLE_SECURE_COOKIES: z.string().default("true").transform(booleanTransform),
// ...

这样可以减少代码重复，提高可维护性。

[claude]

安全建议：默认值问题

fallbackToDirect 使用 provider.proxyFallbackToDirect 作为默认值，但如果该字段为 undefined，这里会返回 undefined 而不是 false。

建议修改：

return {
  agent,
  fallbackToDirect: provider.proxyFallbackToDirect ?? false, // 使用 ?? 确保默认为 false
  proxyUrl: maskProxyUrl(proxyUrl),
};

这样可以确保当字段为 undefined 或 null 时，默认不降级（更安全的策略）。

[claude]

UI 改进建议：添加超时提示

当前测试超时时只显示 "连接超时（5秒）"，但用户可能不知道这是正常的代理连接测试超时时间。

建议改进：

toast.error("连接失败", {
  description:
    errorType === "Timeout"
      ? "连接超时（5秒）。请检查：\n1. 代理服务器是否可访问\n2. 代理地址和端口是否正确\n3. 代理认证信息是否正确"
      : errorType === "ProxyError"
        ? `代理错误: ${errorMessage}`
        : `网络错误: ${errorMessage}`,
  duration: 5000, // 延长显示时间，让用户看清楚
});

这样可以提供更友好的用户体验和诊断提示。

[claude]

代码审查总结

感谢你的贡献！这个 PR 包含了两个重要的修复和功能增强：

✅ 主要改进

1. 修复 HTTP 无法登录问题

   • 修复了 ENABLE_SECURE_COOKIES 环境变量解析错误（"false" 字符串被误判为 true）
   • 在 env.schema.ts 中添加了详细的注释说明原因
   • 解决了非 HTTPS 环境下无法登录的问题

2. 新增供应商级别代理支持

   • 支持 HTTP/HTTPS/SOCKS4/SOCKS5 多种协议
   • 提供代理失败降级策略（proxy_fallback_to_direct）
   • 完善的密码脱敏和安全日志
   • UI 提供代理连接测试功能
   • 文档详尽（CLAUDE.md 新增 111 行）

📝 行内评论

我已经在代码中添加了 4 条行内评论，主要关注：

1. 安全性：代理降级逻辑可能导致配置残留
2. 代码质量：布尔值转换逻辑可以提取为公共函数
3. 健壮性：fallbackToDirect 默认值处理
4. 用户体验：测试失败时提供更详细的诊断提示

🔍 其他观察

• 数据库迁移: 正确删除了重复的迁移文件 0013_optimal_darkhawk.sql，新增 0014_overconfident_mongu.sql
• 价格数据更新: litellm-prices.json 有大量变更（22595 新增，24120 删除），这是正常的价格同步
• 类型安全: 所有新增字段都有完整的 TypeScript 类型定义

⚠️ 合并前建议

虽然代码整体质量很高，但建议在合并前考虑以下几点：

1. 处理代理降级逻辑的潜在配置残留问题
2. 确保 fallbackToDirect 默认值为 false（更安全）
3. 如果时间允许，可以提取布尔值转换逻辑为公共函数

总体评价：这是一个高质量的 PR，修复了关键问题并添加了有价值的功能。代码结构清晰，文档完善，测试考虑周全。✨

[claude]

⚠️ 严重问题：PR 目标分支错误

这个 PR 的目标分支是 main，违反了项目的 Git 工作流规范。

必须遵循的工作流：

• 所有代码变更必须先合并到 dev 分支
• 经过测试和验证后，再从 dev 合并到 main
• 禁止直接向 main 提交 PR

解决方案：

1. 关闭当前 PR
2. 重新创建 PR，指定目标分支为 dev：

   gh pr create --base dev --title "fix: 修复 HTTP 无法登录问题；优化部分文案" --body "详细描述变更内容"

请修正后我会继续进行代码审查。

[claude]

📋 代码审查总结

经过详细审查，PR #49 及其后续修复提交（7cdfdf4）实现了一个完整、安全且可维护的代理支持功能。

✅ 主要功能

1. 代理支持：支持 HTTP、HTTPS、SOCKS4、SOCKS5 四种协议
2. 故障降级：可配置代理失败时是否降级到直连
3. 连接测试：提供 UI 测试按钮验证代理配置
4. 安全性：日志中自动脱敏代理密码
5. HTTP Cookie 安全：修复了非 HTTPS 环境下的登录问题

✅ 代码质量亮点

PR #49 修复提交（7cdfdf4）成功应用了 4 项关键改进：

1. 默认值处理 (proxy-agent.ts)

   fallbackToDirect: provider.proxyFallbackToDirect ?? false  // 避免 undefined

2. 消除代码重复 (env.schema.ts)

   • 提取公共 booleanTransform 函数
   • 从 5 处重复逻辑减至 1 处定义
   • 提高可维护性

3. 对象隔离 (forwarder.ts)

   const fallbackInit = { ...init };  // 创建新对象，避免状态污染
   delete fallbackInit.dispatcher;

4. 用户体验改善 (proxy-test-button.tsx)

   • 超时错误提示改为 3 步排查指南
   • Toast 显示时间延长到 5 秒

📊 评分

维度	评分
功能实现	9.5/10
代码质量	9/10
安全性	8.5/10
文档完整性	9.5/10
类型安全	9.5/10
错误处理	9/10
可维护性	9/10

总体评分：9.0/10

🔍 关键审查结果

✅ 安全性

• 代理密码自动脱敏（maskProxyUrl）
• 默认值保守（fallbackToDirect=false）
• Cookie 安全策略文档完整

✅ 类型安全

• TypeScript 类型定义完整
• 数据库 Schema 与类型一致
• Zod 验证规范

✅ 错误处理

• 异常处理完善
• 日志记录详细（包含主机、端口、目标 URL）
• 降级逻辑清晰

✅ 文档完整性

• CLAUDE.md 新增 76 行代理配置指南
• .env.example 包含详细说明和警告
• 故障排查指南清晰

💡 改进建议（可选）

高优先级：

1. 代理证书验证 - 可考虑添加 TLS 验证选项配置
2. 代理连接超时配置 - 当前硬编码 5 秒，可考虑环境变量化

中优先级：
3. 代理性能监控 - 添加代理延迟指标到统计表
4. 代理失败告警 - 集成到现有告警系统

⚠️ 关键问题：PR 目标分支错误

必须修正：当前 PR 指向 main 分支，违反了项目的 Git 工作流规范。

请：

1. 关闭当前 PR
2. 重新创建 PR 指向 dev 分支：

   gh pr create --base dev --title "fix: 修复 HTTP 无法登录问题；优化部分文案" --body "详细描述..."

✅ 结论

代码质量优秀，功能完整，安全考虑周全，文档详细。修正目标分支后建议合并。