walinejs / waline

💬 A Simple, Safe Comment System
https://waline.js.org/en/
GNU General Public License v2.0
2.27k stars 397 forks source link

[Bug]: `AVATAR_PROXY` 设置后前端崩溃 #990

Closed edgexyz closed 2 years ago

edgexyz commented 2 years ago

问题描述 | Describe the bug

因为 workers.dev 域名国内存在间歇性 sni 阻断的情况,我打算利用自己的域名 libravatar.icesh.me 对项目组提供的 avatar.75cdn.workers.dev 进行反代。

反代域名测试成功,但是在 Vercel 上设置 environment variable AVATAR_PROXY 后 deploy log 显示成功,deployment error log 报错 undefined ERROR TypeError: Cannot read property 'split' of undefined,删除 AVATAR_PROXY 后恢复正常

问题网站 | Website URL

comments.icesh.me

服务部署在哪里? | Where your waline deploy?

Vercel (Default)

数据存储在哪里?| Where your comment data store?

LeanCloud(https://leancloud.app)

edgexyz commented 2 years ago

看了一下源码,怀疑是 config.js 里没有将 AVATAR_PROXY 用单引号包裹起来?

let avatarProxy = 'https://avatar.75cdn.workers.dev/';
if (AVATAR_PROXY) {
  avatarProxy = !isFalse(AVATAR_PROXY) ? AVATAR_PROXY : '';
}
edgexyz commented 2 years ago

vercel environment variable 设置为: NAME: AVATAR_PROXY VALUE 设置为带单引号 'libravatar.icesh.me' 和不带的形式均构建失败

lizheming commented 2 years ago

@edgexyz 剛才有個發版有點小問題,剛發了 1.15.1 的版本你更新下在看一下。另外 AVATAR_PROXY 需要帶 https:// 前綴

edgexyz commented 2 years ago

问题解决了,非常感谢。