From 140d45147205c3a78c16b6b184f685470b8c5974 Mon Sep 17 00:00:00 2001
From: eoao <maillab@skymail.ink>
Date: Tue, 21 Oct 2025 20:45:20 +0800
Subject: [PATCH] =?UTF-8?q?=E6=96=B0=E5=A2=9EKV=E6=95=B0=E6=8D=AE=E5=BA=93?=
 =?UTF-8?q?=E5=AD=98=E5=82=A8=E9=99=84=E4=BB=B6?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../src/components/email-scroll/index.vue     |  4 +-
 mail-vue/src/i18n/en.js                       | 21 +++--
 mail-vue/src/i18n/zh.js                       |  6 +-
 mail-vue/src/layout/account/index.vue         |  5 +-
 mail-vue/src/request/setting.js               |  6 +-
 mail-vue/src/views/sys-setting/index.vue      | 92 ++++++++++++-------
 mail-worker/src/api/r2-api.js                 |  4 +-
 mail-worker/src/api/setting-api.js            |  6 ++
 mail-worker/src/const/entity-const.js         |  8 ++
 mail-worker/src/entity/setting.js             |  4 +-
 mail-worker/src/i18n/en.js                    |  1 +
 mail-worker/src/i18n/zh.js                    |  1 +
 mail-worker/src/index.js                      |  8 +-
 mail-worker/src/init/init.js                  | 12 +++
 mail-worker/src/security/security.js          |  5 +-
 mail-worker/src/service/kv-obj-service.js     | 36 ++++++++
 mail-worker/src/service/r2-service.js         | 28 +++++-
 mail-worker/src/service/setting-service.js    | 39 +++++---
 18 files changed, 213 insertions(+), 73 deletions(-)
 create mode 100644 mail-worker/src/service/kv-obj-service.js

diff --git a/mail-vue/src/components/email-scroll/index.vue b/mail-vue/src/components/email-scroll/index.vue
index acaa455..eb91acb 100644
--- a/mail-vue/src/components/email-scroll/index.vue
+++ b/mail-vue/src/components/email-scroll/index.vue
@@ -494,8 +494,8 @@ function getEmailList(refresh = false) {
   props.getEmailList(queryParam.emailId, queryParam.size).then(async data => {
     let end = Date.now();
     let duration = end - start;
-    if (duration < 500 && !queryParam.emailId) {
-        await sleep(500 - duration)
+    if (duration < 300 && !queryParam.emailId) {
+        await sleep(300 - duration)
     }
     firstLoad.value = false
 
diff --git a/mail-vue/src/i18n/en.js b/mail-vue/src/i18n/en.js
index d279efa..029f42b 100644
--- a/mail-vue/src/i18n/en.js
+++ b/mail-vue/src/i18n/en.js
@@ -32,7 +32,7 @@ const en = {
     username: 'Username',
     password: 'Password',
     delAccount: 'Delete Account',
-    delAccountMsg: 'This will permanently delete your account and data. It cannot be reactivated.',
+    delAccountMsg: 'This will permanently delete your account and data. It cannot be reactivated',
     totalReceived: 'Total Received',
     totalSent: 'Total Sent',
     totalMailboxes: 'Total Accounts',
@@ -137,7 +137,7 @@ const en = {
     websiteReg: 'Sign Up',
     loginDomain: 'Sign-In Box Domain',
     multipleEmail: 'Multiple Accounts',
-    multipleEmailDesc: 'Enable this feature to allow users to add multiple accounts.',
+    multipleEmailDesc: 'Enable this feature to allow users to add multiple accounts',
     customization: 'Customization',
     websiteTitle: 'Title',
     loginBoxOpacity: 'Login Box Opacity',
@@ -145,7 +145,7 @@ const en = {
     emailSetting: 'Email',
     receiveEmail: 'Receive Email',
     autoRefresh: 'Auto Refresh',
-    autoRefreshDesc: 'Automatically fetch the latest emails from the server.',
+    autoRefreshDesc: 'Automatically fetch the latest emails from the server',
     sendEmail: 'Send Email',
     resendToken: 'Resend Token',
     oss: 'Object Storage',
@@ -165,7 +165,7 @@ const en = {
     version: 'Version',
     community: 'Community',
     changeTitle: 'Change Title',
-    addResendTokenDesc: 'Input to add; leave empty to delete.',
+    addResendTokenDesc: 'Input to add; leave empty to delete',
     addOsDomain: 'Add Domain',
     domainDesc: 'Domain',
     addTurnstileSecret: 'Add turnstile secret',
@@ -173,14 +173,14 @@ const en = {
     tgBotDesc: 'Forward received emails to a Telegram bot',
     tgBotToken: 'Bot token',
     toBotTokenDesc: 'Multiple user chat_ids, separated by commas',
-    otherEmailDesc: 'emails can be forwarded to external email, but must be verified via cloudflare.',
+    otherEmailDesc: 'emails can be forwarded to external email, but must be verified via cloudflare',
     otherEmailInputDesc: 'Separate multiple email addresses with commas.',
-    forwardingRulesDesc: 'Rule-based forwarding only forwards emails received by the specified address.',
-    ruleEmailsInputDesc: 'Separate multiple email addresses with commas.',
+    forwardingRulesDesc: 'Rule-based forwarding only forwards emails received by the specified address',
+    ruleEmailsInputDesc: 'Separate multiple email addresses with commas',
     resendTokenList: 'Token List',
     domain: 'Domain',
     optional: 'Optional',
-    subjectInputDesc: 'Please enter the email subject.',
+    subjectInputDesc: 'Please enter the email subject',
     changeUserName: 'Change Username',
     sendSeparately: 'Separately',
     send: 'Send',
@@ -294,7 +294,10 @@ const en = {
     s3Configuration: 'S3 Configuration',
     confirmDeletionOfContacts: 'Confirm clearing contacts?',
     recentContacts: 'Recent contacts',
-    selectContacts: 'Select'
+    selectContacts: 'Select',
+    forcePathStyleDesc: 'Some self-hosted object storages require path-style access to be enabled',
+    kvStorageDesc: 'Replace object storage with KV, and update the access domain to a Worker custom domain',
+    kvStorage: 'KV Storage'
 }
 
 export default en
diff --git a/mail-vue/src/i18n/zh.js b/mail-vue/src/i18n/zh.js
index 757b602..6622b52 100644
--- a/mail-vue/src/i18n/zh.js
+++ b/mail-vue/src/i18n/zh.js
@@ -292,8 +292,12 @@ const zh = {
     include: '包含',
     delAllEmailConfirm: '确定要删除吗？',
     s3Configuration: 'S3 配置',
+    s3Desc: '设置其他S3协议存储，如果绑定了R2会优先用R2',
     confirmDeletionOfContacts: '确认清除这些联系人吗？',
     recentContacts: '最近联系人',
-    selectContacts: '选中'
+    selectContacts: '选中',
+    forcePathStyleDesc: '路径样式访问，一些自建的对象存储需要打开',
+    kvStorageDesc: '使用KV替代对象存储，访问域名改成worker自定义域',
+    kvStorage: 'KV存储'
 }
 export default zh
diff --git a/mail-vue/src/layout/account/index.vue b/mail-vue/src/layout/account/index.vue
index 922d52e..7c3288d 100644
--- a/mail-vue/src/layout/account/index.vue
+++ b/mail-vue/src/layout/account/index.vue
@@ -343,9 +343,8 @@ function getAccountList() {
 
     let end = Date.now();
     let duration = end - start;
-    if (duration < 500) {
-      await sleep(500 - duration)
-
+    if (duration < 300) {
+      await sleep(300 - duration)
     }
 
     if (list.length < queryParams.size) {
diff --git a/mail-vue/src/request/setting.js b/mail-vue/src/request/setting.js
index 041441f..9a5f9b0 100644
--- a/mail-vue/src/request/setting.js
+++ b/mail-vue/src/request/setting.js
@@ -16,6 +16,6 @@ export function setBackground(background) {
     return http.put('/setting/setBackground',{background})
 }
 
-export function physicsDeleteAll() {
-    return http.delete('/setting/physicsDeleteAll')
-}
\ No newline at end of file
+export function deleteBackground() {
+    return http.delete('/setting/deleteBackground')
+}
diff --git a/mail-vue/src/views/sys-setting/index.vue b/mail-vue/src/views/sys-setting/index.vue
index 2eddbf6..eb214ce 100644
--- a/mail-vue/src/views/sys-setting/index.vue
+++ b/mail-vue/src/views/sys-setting/index.vue
@@ -197,13 +197,30 @@
                 </div>
               </div>
               <div class="setting-item">
-                <div><span>{{ $t('s3Configuration') }}</span></div>
+                <div>
+                  <span>{{ $t('s3Configuration') }}</span>
+                  <el-tooltip effect="dark" :content="$t('s3Desc')">
+                    <Icon class="warning" icon="fe:warning" width="18" height="18"/>
+                  </el-tooltip>
+                </div>
                 <div class="r2domain">
                   <el-button class="opt-button" size="small" type="primary" @click="addS3Show = true">
                     <Icon icon="fluent:settings-48-regular" width="16" height="16"/>
                   </el-button>
                 </div>
               </div>
+              <div class="setting-item">
+                <div>
+                  <span>{{ $t('kvStorage') }}</span>
+                  <el-tooltip effect="dark" :content="$t('kvStorageDesc')">
+                    <Icon class="warning" icon="fe:warning" width="18" height="18"/>
+                  </el-tooltip>
+                </div>
+                <div class="r2domain">
+                  <el-switch @change="change" :before-change="beforeChange" :active-value="0" :inactive-value="1"
+                             v-model="setting.kvStorage"/>
+                </div>
+              </div>
             </div>
           </div>
 
@@ -636,7 +653,17 @@
           <el-input class="dialog-input" type="text" placeholder="Region" v-model="s3.region"/>
           <el-input class="dialog-input" type="text" :placeholder="setting.s3AccessKey || 'Access Key'"
                     v-model="s3.s3AccessKey"/>
-          <el-input type="text" :placeholder="setting.s3SecretKey || 'Secret Key'" v-model="s3.s3SecretKey"/>
+          <el-input style="margin-bottom: 10px" type="text" :placeholder="setting.s3SecretKey || 'Secret Key'" v-model="s3.s3SecretKey"/>
+          <div class="force-path-style">
+            <div class="force-path-style-left">
+              <span>ForcePathStyle</span>
+              <el-tooltip effect="dark" :content="$t('forcePathStyleDesc')">
+                <Icon class="warning" icon="fe:warning" width="18" height="18"/>
+              </el-tooltip>
+            </div>
+            <el-switch :before-change="beforeChange" :active-value="0" :inactive-value="1"
+                       v-model="s3.forcePathStyle"/>
+          </div>
           <div class="s3-button">
             <el-button :loading="clearS3Loading" @click="clearS3">{{ t('clear') }}</el-button>
             <el-button type="primary" :loading="settingLoading && !clearS3Loading" @click="saveS3">{{ t('save') }}</el-button>
@@ -649,7 +676,7 @@
 
 <script setup>
 import {computed, defineOptions, reactive, ref} from "vue";
-import {physicsDeleteAll, setBackground, settingQuery, settingSet} from "@/request/setting.js";
+import {deleteBackground, setBackground, settingQuery, settingSet} from "@/request/setting.js";
 import {useSettingStore} from "@/store/setting.js";
 import {useUiStore} from "@/store/ui.js";
 import {useUserStore} from "@/store/user.js";
@@ -718,7 +745,8 @@ const s3 = reactive({
   endpoint: '',
   region: '',
   s3AccessKey: '',
-  s3SecretKey: ''
+  s3SecretKey: '',
+  forcePathStyle: 1
 })
 
 const noticeForm = reactive({
@@ -799,6 +827,7 @@ function resetAddS3Form() {
   s3.region = setting.value.region
   s3.s3AccessKey = ''
   s3.s3SecretKey = ''
+  s3.forcePathStyle = setting.value.forcePathStyle
 }
 
 const resendList = computed(() => {
@@ -977,7 +1006,8 @@ function clearS3() {
     endpoint: '',
     region: '',
     s3AccessKey: '',
-    s3SecretKey: ''
+    s3SecretKey: '',
+    forcePathStyle: 1
   }
   clearS3Loading.value = true
   editSetting(form)
@@ -988,7 +1018,8 @@ function saveS3() {
   const form = {
     bucket: s3.bucket,
     endpoint: s3.endpoint,
-    region: s3.region
+    region: s3.region,
+    forcePathStyle: s3.forcePathStyle
   }
 
   if (s3.s3AccessKey) form.s3AccessKey = s3.s3AccessKey
@@ -1034,35 +1065,21 @@ const opacityChange = debounce(doOpacityChange, 1000, {
   trailing: true
 })
 
-function physicsDeleteAllData() {
-  ElMessageBox.prompt(t('clearAllDelConfirm'), {
-    confirmButtonText: t('confirm'),
-    cancelButtonText: t('cancel'),
-    dangerouslyUseHTMLString: true,
-    title: t('warning'),
-    type: 'warning',
-    inputPattern: new RegExp(`^${t('delInputPattern')}$`),
-    inputErrorMessage: t('inputErrorMessage'),
-  }).then(() => {
-    physicsDeleteAll().then(() => {
-      ElMessage({
-        message: t('delSuccessMsg'),
-        type: "success",
-        plain: true
-      })
-    })
-  })
-}
-
 function delBackground() {
   ElMessageBox.confirm(t('delBackgroundConfirm'), {
     confirmButtonText: t('confirm'),
     cancelButtonText: t('cancel'),
     type: 'warning'
   }).then(() => {
-    backgroundUrl.value = ''
-    setting.value.background = null
-    editSetting({background: null})
+    deleteBackground().then(() => {
+      backgroundUrl.value = ''
+      setting.value.background = null
+      ElMessage({
+        message: t('delSuccessMsg'),
+        type: "success",
+        plain: true
+      })
+    })
   })
 }
 
@@ -1368,7 +1385,7 @@ function editSetting(settingForm, refreshStatus = true) {
 }
 
 .warning {
-  margin-left: 4px;
+  margin-left: 2px;
   color: grey;
   cursor: pointer;
 }
@@ -1595,6 +1612,19 @@ function editSetting(settingForm, refreshStatus = true) {
   margin-bottom: 15px;
 }
 
+.force-path-style {
+  display: flex;
+  align-items: center;
+  justify-content: space-between;
+  gap: 10px;
+  .force-path-style-left {
+    display: flex;
+    justify-content: center;
+    align-items: center;
+    gap: 3px;
+  }
+}
+
 .concerning-item {
   display: flex;
   align-items: center;
@@ -1648,7 +1678,7 @@ function editSetting(settingForm, refreshStatus = true) {
 }
 
 form .el-button {
-  margin-top: 15px;
+  margin-top: 10px;
   width: 100%;
 }
 
diff --git a/mail-worker/src/api/r2-api.js b/mail-worker/src/api/r2-api.js
index 7c33958..9ad07be 100644
--- a/mail-worker/src/api/r2-api.js
+++ b/mail-worker/src/api/r2-api.js
@@ -1,8 +1,8 @@
 import r2Service from '../service/r2-service';
 import app from '../hono/hono';
 
-app.get('/file/*', async (c) => {
-	const key = c.req.path.split('/file/')[1];
+app.get('/oss/*', async (c) => {
+	const key = c.req.path.split('/oss/')[1];
 	const obj = await r2Service.getObj(c, key);
 	return new Response(obj.body, {
 		headers: {
diff --git a/mail-worker/src/api/setting-api.js b/mail-worker/src/api/setting-api.js
index 16409ac..298621e 100644
--- a/mail-worker/src/api/setting-api.js
+++ b/mail-worker/src/api/setting-api.js
@@ -21,3 +21,9 @@ app.put('/setting/setBackground', async (c) => {
 	const key = await settingService.setBackground(c, await c.req.json());
 	return c.json(result.ok(key));
 });
+
+app.delete('/setting/deleteBackground', async (c) => {
+	await settingService.deleteBackground(c);
+	return c.json(result.ok());
+});
+
diff --git a/mail-worker/src/const/entity-const.js b/mail-worker/src/const/entity-const.js
index ee9f894..550ac79 100644
--- a/mail-worker/src/const/entity-const.js
+++ b/mail-worker/src/const/entity-const.js
@@ -107,6 +107,14 @@ export const settingConst = {
 	noRecipient: {
 		OPEN: 0,
 		CLOSE: 1,
+	},
+	kvStorage: {
+		OPEN: 0,
+		CLOSE: 1
+	},
+	forcePathStyle: {
+		OPEN: 0,
+		CLOSE: 1
 	}
 }
 
diff --git a/mail-worker/src/entity/setting.js b/mail-worker/src/entity/setting.js
index 649806f..ed42ece 100644
--- a/mail-worker/src/entity/setting.js
+++ b/mail-worker/src/entity/setting.js
@@ -39,6 +39,8 @@ export const setting = sqliteTable('setting', {
 	region: text('region').default('').notNull(),
 	endpoint: text('endpoint').default('').notNull(),
 	s3AccessKey: text('s3_access_key').default('').notNull(),
-	s3SecretKey: text('s3_secret_key').default('').notNull()
+	s3SecretKey: text('s3_secret_key').default('').notNull(),
+	kvStorage: integer('kv_storage').default(1).notNull(),
+	forcePathStyle: integer('force_path_style').default(1).notNull()
 });
 export default setting
diff --git a/mail-worker/src/i18n/en.js b/mail-worker/src/i18n/en.js
index 6e5ca33..b69a0e2 100644
--- a/mail-worker/src/i18n/en.js
+++ b/mail-worker/src/i18n/en.js
@@ -63,6 +63,7 @@ const en = {
 	publicTokenFail: 'Token validation failed',
 	notAdmin: 'The entered email is not an administrator email',
 	emailExistDatabase: 'Email already exists in the database',
+	notConfigOss: 'Object storage not configured',
 	perms: {
 		"邮件": "Email",
 		"邮件发送": "Send Email",
diff --git a/mail-worker/src/i18n/zh.js b/mail-worker/src/i18n/zh.js
index 23b8838..cc991a8 100644
--- a/mail-worker/src/i18n/zh.js
+++ b/mail-worker/src/i18n/zh.js
@@ -63,6 +63,7 @@ const zh = {
 	publicTokenFail: 'token验证失败',
 	notAdmin: '输入的邮箱不是管理员邮箱',
 	emailExistDatabase: '有邮箱已存在数据库中',
+	notConfigOss: '对象存储未配置',
 	perms: {
 		"邮件": "邮件",
 		"邮件发送": "邮件发送",
diff --git a/mail-worker/src/index.js b/mail-worker/src/index.js
index 8b386a0..77bd535 100644
--- a/mail-worker/src/index.js
+++ b/mail-worker/src/index.js
@@ -3,10 +3,11 @@ import { email } from './email/email';
 import userService from './service/user-service';
 import verifyRecordService from './service/verify-record-service';
 import emailService from './service/email-service';
+import kvObjService from './service/kv-obj-service';
 export default {
 	 async fetch(req, env, ctx) {
-		const url = new URL(req.url)
 
+		const url = new URL(req.url)
 
 		if (url.pathname.startsWith('/api/')) {
 			url.pathname = url.pathname.replace('/api', '')
@@ -14,12 +15,15 @@ export default {
 			return app.fetch(req, env, ctx);
 		}
 
+		 if (['/static/','/attachments/'].some(p => url.pathname.startsWith(p))) {
+			 return await kvObjService.toObjResp( { env }, url.pathname.substring(1));
+		 }
 
 		return env.assets.fetch(req);
 	},
 	email: email,
 	async scheduled(c, env, ctx) {
-		await verifyRecordService.clearRecord({env})
+		await verifyRecordService.clearRecord({ env })
 		await userService.resetDaySendCount({ env })
 		await emailService.completeReceiveAll({ env })
 	},
diff --git a/mail-worker/src/init/init.js b/mail-worker/src/init/init.js
index 69aae93..f09ac45 100644
--- a/mail-worker/src/init/init.js
+++ b/mail-worker/src/init/init.js
@@ -22,10 +22,22 @@ const init = {
 		await this.v1_6DB(c);
 		await this.v1_7DB(c);
 		await this.v2DB(c);
+		await this.v2_3DB(c);
 		await settingService.refresh(c);
 		return c.text(t('initSuccess'));
 	},
 
+	async v2_3DB(c) {
+		try {
+			await c.env.db.batch([
+				c.env.db.prepare(`ALTER TABLE setting ADD COLUMN force_path_style	INTEGER NOT NULL DEFAULT 1;`),
+				c.env.db.prepare(`ALTER TABLE setting ADD COLUMN kv_storage INTEGER NOT NULL DEFAULT 1;`)
+			]);
+		} catch (e) {
+			console.error(e.message)
+		}
+	},
+
 	async v2DB(c) {
 		try {
 			await c.env.db.batch([
diff --git a/mail-worker/src/security/security.js b/mail-worker/src/security/security.js
index 6fa3ca4..1dcd790 100644
--- a/mail-worker/src/security/security.js
+++ b/mail-worker/src/security/security.js
@@ -11,7 +11,7 @@ import app from '../hono/hono';
 const exclude = [
 	'/login',
 	'/register',
-	'/file',
+	'/oss',
 	'/setting/websiteConfig',
 	'/webhooks',
 	'/init',
@@ -34,6 +34,7 @@ const requirePerms = [
 	'/allEmail/list',
 	'/allEmail/delete',
 	'/setting/setBackground',
+	'/setting/deleteBackground',
 	'/setting/set',
 	'/setting/query',
 	'/user/delete',
@@ -73,7 +74,7 @@ const premKey = {
 	'all-email:query': ['/allEmail/list'],
 	'all-email:delete': ['/allEmail/delete','/allEmail/batchDelete'],
 	'setting:query': ['/setting/query'],
-	'setting:set': ['/setting/set', '/setting/setBackground'],
+	'setting:set': ['/setting/set', '/setting/setBackground','/setting/deleteBackground'],
 	'analysis:query': ['/analysis/echarts'],
 	'reg-key:add': ['/regKey/add'],
 	'reg-key:query': ['/regKey/list','/regKey/history'],
diff --git a/mail-worker/src/service/kv-obj-service.js b/mail-worker/src/service/kv-obj-service.js
new file mode 100644
index 0000000..965d2d5
--- /dev/null
+++ b/mail-worker/src/service/kv-obj-service.js
@@ -0,0 +1,36 @@
+const kvObjService = {
+
+	async putObj(c, key, content, metadata) {
+		await c.env.kv.put(key, content, { metadata: metadata });
+	},
+
+	async deleteObj(c, keys) {
+
+		if (typeof keys === 'string') {
+			keys = [keys];
+		}
+
+		if (keys.length === 0) {
+			return;
+		}
+
+		await Promise.all(keys.map( key => c.env.kv.delete(key)));
+	},
+
+	async toObjResp(c, key) {
+
+		const obj = await c.env.kv.getWithMetadata(key, { type: "arrayBuffer"});
+
+		return new Response(obj.value, {
+			headers: {
+				'Content-Type': obj.metadata?.contentType || 'application/octet-stream',
+				'Content-Disposition': obj.metadata?.contentDisposition || null,
+				'Cache-Control': obj.metadata?.cacheControl || null
+			}
+		});
+
+	}
+
+};
+
+export default kvObjService;
diff --git a/mail-worker/src/service/r2-service.js b/mail-worker/src/service/r2-service.js
index 523e48c..f9e9aaa 100644
--- a/mail-worker/src/service/r2-service.js
+++ b/mail-worker/src/service/r2-service.js
@@ -1,23 +1,35 @@
 import s3Service from './s3-service';
 import settingService from './setting-service';
+import kvObjService from './kv-obj-service';
+import { settingConst } from '../const/entity-const';
 
 const r2Service = {
 
 	async hasOSS(c) {
 
-		if (c.env.r2) {
+		const setting = await settingService.query(c);
+		const { kvStorage, bucket, endpoint, s3AccessKey, s3SecretKey } = setting;
+
+		if (kvStorage === settingConst.kvStorage.OPEN) {
 			return true;
 		}
 
-		const setting = await settingService.query(c);
-		const { bucket, endpoint, s3AccessKey, s3SecretKey } = setting;
+		if (c.env.r2) {
+			return true;
+		}
 
 		return !!(bucket && endpoint && s3AccessKey && s3SecretKey);
 	},
 
 	async putObj(c, key, content, metadata) {
 
-		if (c.env.r2) {
+		const { kvStorage } = await settingService.query(c);
+
+		if (kvStorage === settingConst.kvStorage.OPEN) {
+
+			await kvObjService.putObj(c, key, content, metadata);
+
+		} else if (c.env.r2) {
 
 			await c.env.r2.put(key, content, {
 				httpMetadata: { ...metadata }
@@ -37,7 +49,13 @@ const r2Service = {
 
 	async delete(c, key) {
 
-		if (c.env.r2) {
+		const { kvStorage } = await settingService.query(c);
+
+		if (kvStorage === settingConst.kvStorage.OPEN) {
+
+			await kvObjService.deleteObj(c, key);
+
+		} else if (c.env.r2) {
 
 			await c.env.r2.delete(key);
 
diff --git a/mail-worker/src/service/setting-service.js b/mail-worker/src/service/setting-service.js
index 92a9a78..9dc9922 100644
--- a/mail-worker/src/service/setting-service.js
+++ b/mail-worker/src/service/setting-service.js
@@ -1,12 +1,9 @@
 import KvConst from '../const/kv-const';
 import setting from '../entity/setting';
 import orm from '../entity/orm';
-import { settingConst, verifyRecordType } from '../const/entity-const';
+import { verifyRecordType } from '../const/entity-const';
 import fileUtils from '../utils/file-utils';
 import r2Service from './r2-service';
-import emailService from './email-service';
-import accountService from './account-service';
-import userService from './user-service';
 import constant from '../const/constant';
 import BizError from '../error/biz-error';
 import { t } from '../i18n/i18n'
@@ -100,12 +97,38 @@ const settingService = {
 		await this.refresh(c);
 	},
 
+	async deleteBackground(c) {
+
+		const { background } = await this.query(c);
+		if (!background) return
+
+		if (background.startsWith('http')) {
+			await orm(c).update(setting).set({ background: '' }).run();
+			await this.refresh(c)
+			return;
+		}
+
+		const hasOss = await r2Service.hasOSS(c);
+
+		if (hasOss) {
+
+			if (background) {
+				await r2Service.delete(c,background)
+				await orm(c).update(setting).set({ background: '' }).run();
+				await this.refresh(c)
+			}
+
+		}
+	},
+
 	async setBackground(c, params) {
 
 		const settingRow = await this.query(c);
 
 		let { background } = params
 
+		await this.deleteBackground(c);
+
 		if (background && !background.startsWith('http')) {
 
 			if (!await r2Service.hasOSS(c)) {
@@ -130,14 +153,6 @@ const settingService = {
 
 		}
 
-		if (settingRow.background) {
-			try {
-				await r2Service.delete(c, settingRow.background);
-			} catch (e) {
-				console.error(e)
-			}
-		}
-
 		await orm(c).update(setting).set({ background }).run();
 		await this.refresh(c);
 		return background;