修复:全站图片 503 —— oss.enabled 派生写入被 has() 守卫拦掉

问题:attachment.storage 存成 oss,但 oss.enabled 一直停在 0。
media.php 判 fb_oss_enabled() 为 false,直接 503「对象存储未配置」,
所有 OSS 附件(历史 + 新增)全部打不开。

根因有两层:
1. oss.enabled 是派生字段,表单里没有对应 input,上一个提交加的
   $save() 统一守卫用 $has() 判断「没提交」就 return,把它挡掉了;
2. 更根本的是两份状态描述同一件事 —— storage 和 enabled 迟早漂移。

修法:
- 派生写入改用 $saveForce(),并整段包进 if ($has(attachment.storage)),
  避免反向问题(没提交存储方式时被默认值改成 off);
- fb_oss_enabled() 改为只看 attachment.storage 作单一事实来源,
  oss.enabled 仅在 storage 完全没配过的老库上作为回落。
This commit is contained in:
2026-11-03 07:21:04 +08:00
parent a850d479f0
commit 8a850ca0e5
2 changed files with 35 additions and 6 deletions
+17 -5
View File
@@ -453,11 +453,23 @@ function fb_admin_save_settings() {
$save($k, (int)round($mb * 1048576)); $save($k, (int)round($mb * 1048576));
} }
$storage = (string)$post('attachment.storage', 'off'); // 附件存储方式:只有表单真的带了这个字段才处理。
if (!in_array($storage, ['off', 'local', 'oss'], true)) { $storage = 'off'; } // 没有它(比如只提交了 site_name 的局部表单)就整段跳过 ——
$save('attachment.storage', $storage); // 否则下面 $post(..., 'off') 的默认值会把存储方式误改成「关闭」。
// OSS 模式下 oss.enabled 必须跟着开;其它模式关掉,避免误判 if ($has('attachment.storage')) {
$save('oss.enabled', $storage === 'oss' ? 1 : 0); $storage = (string)$post('attachment.storage', 'off');
if (!in_array($storage, ['off', 'local', 'oss'], true)) { $storage = 'off'; }
$save('attachment.storage', $storage);
// OSS 模式下 oss.enabled 必须跟着开;其它模式关掉,避免误判。
// 🔴 这里必须用 $saveForce,不能用 $save:
// oss.enabled 是【派生字段】—— 表单里根本没有名为 oss_enabled 的 input,
// $save 的 $has() 守卫会判定"没提交"而直接 return,于是 storage 存成了 oss、
// enabled 却永远停在旧值 0。media.php 因此报「对象存储未配置」,全站图片 503。
// (2026-11-03 图片全挂的直接原因:守卫修好了跨 tab 清空,却把派生写入也挡掉了。)
// 又因为它处在 $has('attachment.storage') 分支内,不会出现「没提交却强制写 0」的反向问题。
$saveForce('oss.enabled', $storage === 'oss' ? 1 : 0);
}
$texts = ['oss.endpoint', 'oss.bucket', 'oss.instance_role', 'oss.domain', 'oss.prefix', 'moderation.close_mail_subject']; $texts = ['oss.endpoint', 'oss.bucket', 'oss.instance_role', 'oss.domain', 'oss.prefix', 'moderation.close_mail_subject'];
foreach ($texts as $k) { foreach ($texts as $k) {
+18 -1
View File
@@ -34,7 +34,24 @@ function fb_oss_cfg($k = null, $d = null) {
*/ */
function fb_oss_enabled() { function fb_oss_enabled() {
$c = fb_oss_cfg(); $c = fb_oss_cfg();
if (empty($c['enabled'])) { return false; }
/* 🔴 启用判据只看 attachment.storage(单一事实来源)。
*
* 原先这里只看 oss.enabled —— 而 oss.enabled 是个【派生字段】:
* 后台表单里没有对应的 input,只能由保存逻辑“顺手”写一次。
* 两份状态描述同一件事,就必然有漂移的一天,实际也真的漂了:
* 2026-11-03 attachment.storage=oss 而 oss.enabled=0,
* 结果上传/读取全被判成「对象存储未配置」,全站图片 503。
*
* 现在只有 storage 完全没配过(老库、安装向导中途)才回落到 oss.enabled。
*/
$storage = strtolower(trim((string)fb_cfg('attachment.storage', '')));
if ($storage !== '') {
if ($storage !== 'oss') { return false; } // local / off → OSS 一律不启用
} elseif (empty($c['enabled'])) {
return false;
}
foreach (['bucket', 'endpoint'] as $k) { foreach (['bucket', 'endpoint'] as $k) {
if (trim((string)(isset($c[$k]) ? $c[$k] : '')) === '') { return false; } if (trim((string)(isset($c[$k]) ? $c[$k] : '')) === '') { return false; }
} }