:art: 改进数据仓库密钥初始化确认过程

app/appearance/langs/en_US.json

@@ -1,4 +1,5 @@
 {
+  "initRepoKeyTip": "If the key has been initialized on other devices, be sure to use the [Import key] function, otherwise the data cannot be synced to the cloud, so be sure to use the same key on all devices",
   "crossKeepLazyLoad": "Cross-page multi-selection needs to select [Keep Loaded Content] in the more menu",
   "keepLazyLoad": "Keep loaded content",
   "localSnapshot": "Local snapshot",
@@ -30,7 +31,7 @@
   "key": "Key",
   "genKey": "Generate key",
   "dataRepoKeyTip1": "The data repo is used to encrypt and save data snapshots. Snapshots can be created and rolled back in the data history",
-  "dataRepoKeyTip2": "After the key is initialized on the main device, other devices need to import the key. If the device keys are inconsistent, data snapshots cannot be synced to the cloud, so be sure to use the same key on all devices",
+  "dataRepoKeyTip2": "After the key is initialized, other devices need to import the key. If the device keys don't match, you won't be able to sync data in the cloud, so be sure to use the same key on all devices",
   "dataRepoKey": "Data repo key",
   "plsChoose": "Please choose first",
   "clearMessage": "Read",
@@ -909,7 +910,7 @@
     "134": "In order to prevent the newly restored data from being overwritten by synchronization, the data synchronization function has been automatically suspended",
     "135": "Please make sure that all devices have been updated to the latest version, and then trigger synchronization after randomly changing a document on the main device, and finally trigger synchronization on other devices",
     "136": "Initializing data repository key...",
-    "137": "Failed to initialize data repository key",
+    "137": "Failed to initialize data repository key: %s",
     "138": "Data repository key is set",
     "139": "Data snapshot is being generated, [%s] data files have been processed, [%s] remaining...",
     "140": "Failed to create data snapshot: %s",
@@ -928,6 +929,7 @@
     "153": "Downloaded files %d, chunks %d, received bytes %s",
     "154": "The cloud can only support backup up to 12 snapshots",
     "155": "Cloud sync directory has been reset to [main]",
-    "156": "Access authentication failed, please refresh the page"
+    "156": "Access authentication failed, please refresh the page",
+    "157": "The key is not recognized, please confirm that the copied key string is correct"
   }
 }

app/appearance/langs/es_ES.json

@@ -1,4 +1,5 @@
 {
+  "initRepoKeyTip": "Si la clave se ha inicializado en otros dispositivos, asegúrese de usar la función [Importar la clave], de lo contrario, los datos no se pueden sincronizar con la nube, así que asegúrese de usar la misma clave en todos los dispositivos",
   "crossKeepLazyLoad": "La selección múltiple entre páginas debe seleccionar [Mantener contenido cargado] en el menú más",
   "keepLazyLoad": "Mantener el contenido cargado",
   "localSnapshot": "Instantánea local",
@@ -30,7 +31,7 @@
   "key": "Clave",
   "genKey": "Generación de la clave",
   "dataRepoKeyTip1": "El repositorio de datos se utiliza para cifrar y guardar instantáneas de datos. Las instantáneas pueden crearse y revertirse en el historial de datos",
-  "dataRepoKeyTip2": "Una vez inicializada la clave en el dispositivo principal, los demás dispositivos deben importarla. Si las claves de los dispositivos son incoherentes, las instantáneas de datos no se pueden sincronizar con la nube, así que asegúrate de utilizar la misma clave en todos los dispositivos",
+  "dataRepoKeyTip2": "Después de inicializar la clave, otros dispositivos deben importar la clave. Si las claves del dispositivo no coinciden, no podrá sincronizar datos en la nube, así que asegúrese de usar la misma clave en todos los dispositivos",
   "dataRepoKey": "Clave del repositorio de datos",
   "plsChoose": "Por favor, elija primero",
   "clearMessage": "Leer",
@@ -909,7 +910,7 @@
     "134": "Para evitar que los datos recién restaurados sean sobrescritos por la sincronización, se ha suspendido automáticamente la función de sincronización de datos",
     "135": "Por favor, asegúrese de que todos los dispositivos han sido actualizados a la última versión y, a continuación, active la sincronización después de cambiar aleatoriamente un documento en el dispositivo principal y, finalmente, active la sincronización en otros dispositivos",
     "136": "Inicializando la clave del repositorio de datos...",
-    "137": "Fallo en la inicialización de la clave del repositorio de datos",
+    "137": "Fallo en la inicialización de la clave del repositorio de datos: %s",
     "138": "La clave del repositorio de datos está configurada",
     "139": "Se está generando la instantánea de datos, se han procesado [%s] archivos de datos, quedan [%s]...",
     "140": "Fallo en la creación de la instantánea de datos: %s",
@@ -928,6 +929,7 @@
     "153": "Archivos descargados %d, fragmentos %d, bytes recibidos %s",
     "154": "La nube solo admite copias de seguridad de hasta 12 instantáneas",
     "155": "El directorio de sincronización de la nube se ha restablecido a [main]",
-    "156": "Error en la autenticación de acceso, actualice la página"
+    "156": "Error en la autenticación de acceso, actualice la página",
+    "157": "No se reconoce la clave, confirme que la cadena de clave copiada es correcta"
   }
 }

app/appearance/langs/fr_FR.json

@@ -1,4 +1,5 @@
 {
+  "initRepoKeyTip": "If the key has been initialized on other devices, be sure to use the [Importer la clé] function, otherwise the data cannot be synced to the cloud, so be sure to use the same key on all devices",
   "crossKeepLazyLoad": "La multi-sélection sur plusieurs pages doit sélectionner [Conserver le contenu chargé] dans le menu plus",
   "keepLazyLoad": "Conserver le contenu chargé",
   "localSnapshot": "Instantané local",
@@ -30,7 +31,7 @@
   "clé": "Clé",
   "genKey": "Générer la clé",
   "dataRepoKeyTip1": "Le référentiel de données est utilisé pour chiffrer et enregistrer des instantanés de données. Des instantanés peuvent être créés et restaurés dans l'historique des données",
-  "dataRepoKeyTip2": "Une fois la clé initialisée sur l'appareil principal, d'autres appareils doivent importer la clé. Si les clés de l'appareil sont incohérentes, les instantanés de données ne peuvent pas être synchronisés avec le cloud, alors assurez-vous d'utiliser la même clé sur tous les appareils",
+  "dataRepoKeyTip2": "Une fois la clé initialisée, d'autres appareils doivent importer la clé. Si les clés de l'appareil ne correspondent pas, vous ne pourrez pas synchroniser les données dans le cloud, alors assurez-vous d'utiliser la même clé sur tous les appareils",
   "dataRepoKey": "Clé de référentiel de données",
   "plsChoose": "Veuillez d'abord choisir",
   "clearMessage": "Lire",
@@ -909,7 +910,7 @@
     "134": "Afin d'éviter que les données nouvellement restaurées ne soient écrasées par la synchronisation, la fonction de synchronisation des données a été automatiquement suspendue",
     "135": "Assurez-vous que tous les appareils ont été mis à jour vers la dernière version, puis déclenchez la synchronisation après avoir modifié de manière aléatoire un document sur l'appareil principal, et enfin déclenchez la synchronisation sur d'autres appareils.",
     "136": "Initialisation de la clé du référentiel de données...",
-    "137": "Échec de l'initialisation de la clé du référentiel de données",
+    "137": "Échec de l'initialisation de la clé du référentiel de données: %s",
     "138": "La clé du référentiel de données est définie",
     "139": "L'instantané des données est en cours de génération, [%s] fichiers de données ont été traités, [%s] restants...",
     "140": "Échec de la création de l'instantané des données: %s",
@@ -928,6 +929,7 @@
     "153": "Fichiers chargés %d, morceaux %d, octets reçus %s",
     "154": "Le cloud ne peut prendre en charge que la sauvegarde jusqu'à 12 instantanés",
     "155": "Le répertoire de synchronisation du cloud a été réinitialisé sur [main]",
-    "156": "Échec de l'authentification d'accès, veuillez actualiser la page"
+    "156": "Échec de l'authentification d'accès, veuillez actualiser la page",
+    "157": "La clé n'est pas reconnue, veuillez confirmer que la chaîne de clé copiée est correcte"
   }
 }

app/appearance/langs/zh_CHT.json

@@ -1,4 +1,5 @@
 {
+  "initRepoKeyTip": "如果其他設備上已經初始化過密鑰，請務必使用 [導入密鑰] 功能，否則無法雲端同步數據，所以請務必在所有設備上使用相同的密鑰",
   "crossKeepLazyLoad": "跨頁多選需在更多菜單中選中【保持已加載內容】",
   "keepLazyLoad": "保持已加載的內容",
   "localSnapshot": "本地快照",
@@ -30,7 +31,7 @@
   "key": "密鑰",
   "genKey": "生成密鑰",
   "dataRepoKeyTip1": "數據倉庫用於加密保存數據快照，可在數據歷史中可創建快照和回滾快照",
-  "dataRepoKeyTip2": "主力設備上初始化密鑰以後，其他設備需要導入該密鑰。如果設備密鑰不一致，則無法雲端同步數據快照，所以請務必在所有設備上使用相同的密鑰",
+  "dataRepoKeyTip2": "初始化密鑰以後其他設備需要導入該密鑰。如果設備密鑰不一致，則無法雲端同步數據，所以請務必在所有設備上使用相同的密鑰",
   "dataRepoKey": "數據倉庫密鑰",
   "plsChoose": "請先進行選擇",
   "clearMessage": "已讀",
@@ -908,7 +909,7 @@
     "134": "為避免剛恢復的數據被同步覆蓋，數據同步功能已被自動暫停",
     "135": "請確保所有設備已經更新到最新版，然後在主力設備上隨意更改一個文檔後觸發同步，最後再到其他設備觸發同步",
     "136": "初始化數據倉庫密鑰...",
-    "137": "初始化數據倉庫密鑰失敗",
+    "137": "初始化數據倉庫密鑰失敗：%s",
     "138": "數據倉庫密鑰設置完畢",
     "139": "正在生成數據快照，已處理 [%s] 個數據文件，剩餘待處理 [%s]...",
     "140": "創建數據快照失敗：%s",
@@ -927,6 +928,7 @@
     "153": "下載文件數 %d 下載分塊數 %d 接收字節數 %s",
     "154": "雲端最多只能支持備份 12 個快照",
     "155": "雲端同步目錄已經重置為 [main]",
-    "156": "訪問鑑權失敗，請刷新頁面"
+    "156": "訪問鑑權失敗，請刷新頁面",
+    "157": "無法識別密鑰，請確認複製的密鑰字符串是否正確"
   }
 }

app/appearance/langs/zh_CN.json

@@ -1,5 +1,6 @@
 {
-  "crossKeepLazyLoad": "跨页多选需在更多菜单中选中【保持已加载内容】",
+  "initRepoKeyTip": "如果其他设备上已经初始化过密钥，请务必使用 [导入密钥] 功能，否则无法云端同步数据，所以请务必在所有设备上使用相同的密钥",
+  "crossKeepLazyLoad": "跨页多选需在更多菜单中选中 [保持已加载内容]",
   "keepLazyLoad": "保持已加载的内容",
   "localSnapshot": "本地快照",
   "localTagSnapshot": "本地标记的快照",
@@ -30,7 +31,7 @@
   "key": "密钥",
   "genKey": "生成密钥",
   "dataRepoKeyTip1": "数据仓库用于加密保存数据快照，可在数据历史中可创建快照和回滚快照",
-  "dataRepoKeyTip2": "主力设备上初始化密钥以后，其他设备需要导入该密钥。如果设备密钥不一致，则无法云端同步数据快照，所以请务必在所有设备上使用相同的密钥",
+  "dataRepoKeyTip2": "初始化密钥以后其他设备需要导入该密钥。如果设备密钥不一致，则无法云端同步数据，所以请务必在所有设备上使用相同的密钥",
   "dataRepoKey": "数据仓库密钥",
   "plsChoose": "请先进行选择",
   "clearMessage": "已读",
@@ -788,7 +789,7 @@
     "12": "查询资源文件失败 [%s]",
     "13": "无法创建 . 开头的文件",
     "14": "导出失败：%s",
-    "15": "未找到 ID 为 [%s] 的内容块，请在文档树面板菜单【重建索引】以后再试",
+    "15": "未找到 ID 为 [%s] 的内容块，请在文档树面板菜单 [重建索引] 以后再试",
     "16": "请输入文档名",
     "17": "启动时初次同步失败，继续操作可能会发生无法预测的数据覆盖，请先执行一次同步",
     "18": "获取社区用户账号失败",
@@ -910,7 +911,7 @@
     "134": "为避免刚恢复的数据被同步覆盖，数据同步功能已被自动暂停",
     "135": "请确保所有设备已经更新到最新版，然后在主力设备上随意更改一个文档后触发同步，最后再到其他设备触发同步",
     "136": "初始化数据仓库密钥...",
-    "137": "初始化数据仓库密钥失败",
+    "137": "初始化数据仓库密钥失败：%s",
     "138": "数据仓库密钥设置完毕",
     "139": "正在生成数据快照，已处理 [%s] 个数据文件，剩余待处理 [%s]...",
     "140": "创建数据快照失败：%s",
@@ -929,6 +930,7 @@
     "153": "下载文件数 %d 下载分块数 %d 接收字节数 %s",
     "154": "云端最多只能支持备份 12 个快照",
     "155": "云端同步目录已经重置为 [main]",
-    "156": "访问鉴权失败，请刷新页面"
+    "156": "访问鉴权失败，请刷新页面",
+    "157": "无法识别密钥，请确认复制的密钥字符串是否正确"
   }
 }

app/src/config/about.ts

@@ -218,7 +218,7 @@ export const about = {
         const importKeyElement = about.element.querySelector("#importKey");
         importKeyElement.addEventListener("click", () => {
             const passwordDialog = new Dialog({
-                title: window.siyuan.languages.key,
+                title: "🔑 " + window.siyuan.languages.key,
                 content: `<div class="b3-dialog__content">
     <textarea class="b3-text-field fn__block" placeholder="${window.siyuan.languages.keyPlaceholder}"></textarea>
 </div>
@@ -244,7 +244,7 @@ export const about = {
             });
         });
         about.element.querySelector("#initKey").addEventListener("click", () => {
-            confirmDialog(window.siyuan.languages.genKey, "TODO", () => {
+            confirmDialog("🔑 " + window.siyuan.languages.genKey, window.siyuan.languages.initRepoKeyTip, () => {
                 fetchPost("/api/repo/initRepoKey", {}, (response) => {
                     window.siyuan.config.repo.key = response.data.key;
                     importKeyElement.parentElement.classList.add("fn__none");

app/src/mobile/util/menu.ts

@@ -289,7 +289,7 @@ ${accountHTML}
                     const importKeyElement = modelMainElement.querySelector("#importKey");
                     importKeyElement.addEventListener("click", () => {
                         const passwordDialog = new Dialog({
-                            title: window.siyuan.languages.key,
+                            title: "🔑 " + window.siyuan.languages.key,
                             content: `<div class="b3-dialog__content">
     <textarea class="b3-text-field fn__block" placeholder="${window.siyuan.languages.keyPlaceholder}"></textarea>
 </div>
@@ -315,7 +315,7 @@ ${accountHTML}
                         });
                     });
                     modelMainElement.querySelector("#initKey").addEventListener("click", () => {
-                        confirmDialog(window.siyuan.languages.genKey, "TODO", () => {
+                        confirmDialog("🔑 " + window.siyuan.languages.genKey, window.siyuan.languages.initRepoKeyTip, () => {
                             fetchPost("/api/repo/initRepoKey", {}, (response) => {
                                 window.siyuan.config.repo.key = response.data.key;
                                 importKeyElement.parentElement.classList.add("fn__none");

kernel/api/repo.go

@@ -219,7 +219,7 @@ func importRepoKey(c *gin.Context) {
 	base64Key := arg["key"].(string)
 	if err := model.ImportRepoKey(base64Key); nil != err {
 		ret.Code = -1
-		ret.Msg = model.Conf.Language(137)
+		ret.Msg = fmt.Sprintf(model.Conf.Language(137), err)
 		return
 	}
 }
@@ -230,7 +230,7 @@ func initRepoKey(c *gin.Context) {
 
 	if err := model.InitRepoKey(); nil != err {
 		ret.Code = -1
-		ret.Msg = model.Conf.Language(137)
+		ret.Msg = fmt.Sprintf(model.Conf.Language(137), err)
 		return
 	}
 

kernel/model/repository.go

@@ -23,6 +23,7 @@ import (
 	"fmt"
 	"math"
 	"os"
+	"strings"
 	"time"
 
 	"github.com/88250/gulu"
@@ -65,11 +66,19 @@ func GetRepoSnapshots(page int) (logs []*dejavu.Log, pageCount, totalCount int,
 }
 
 func ImportRepoKey(base64Key string) (err error) {
-	msgId := util.PushMsg(Conf.Language(136), 1000*7)
+	util.PushMsg(Conf.Language(136), 3000)
+
+	base64Key = strings.TrimSpace(base64Key)
+	base64Key = gulu.Str.RemoveInvisible(base64Key)
+	if 1 > len(base64Key) {
+		err = errors.New(Conf.Language(142))
+		return
+	}
 
 	key, err := base64.StdEncoding.DecodeString(base64Key)
 	if nil != err {
-		return
+		util.LogErrorf("import data repo key failed: %s", err)
+		return errors.New(Conf.Language(157))
 	}
 	Conf.Repo.Key = key
 	Conf.Save()
@@ -82,10 +91,10 @@ func ImportRepoKey(base64Key string) (err error) {
 	}
 
 	time.Sleep(1 * time.Second)
-	util.PushUpdateMsg(msgId, Conf.Language(138), 3000)
+	util.PushMsg(Conf.Language(138), 3000)
 	time.Sleep(1 * time.Second)
 	if initErr := IndexRepo("[Init] Init data repo"); nil != initErr {
-		util.PushUpdateMsg(msgId, fmt.Sprintf(Conf.Language(140), initErr), 7000)
+		util.PushErrMsg(fmt.Sprintf(Conf.Language(140), initErr), 0)
 	}
 	return
 }
@@ -108,7 +117,7 @@ func ResetRepo() (err error) {
 }
 
 func InitRepoKey() (err error) {
-	msgId := util.PushMsg(Conf.Language(136), 1000*7)
+	util.PushMsg(Conf.Language(136), 3000)
 
 	if err = os.RemoveAll(Conf.Repo.GetSaveDir()); nil != err {
 		return
@@ -127,7 +136,6 @@ func InitRepoKey() (err error) {
 	_, err = rand.Read(randomBytes)
 	if nil != err {
 		util.LogErrorf("init data repo key failed: %s", err)
-		util.PushUpdateMsg(msgId, Conf.Language(137), 5000)
 		return
 	}
 	salt := string(randomBytes)
@@ -135,17 +143,16 @@ func InitRepoKey() (err error) {
 	key, err := encryption.KDF(password, salt)
 	if nil != err {
 		util.LogErrorf("init data repo key failed: %s", err)
-		util.PushUpdateMsg(msgId, Conf.Language(137), 5000)
 		return
 	}
 	Conf.Repo.Key = key
 	Conf.Save()
 
 	time.Sleep(1 * time.Second)
-	util.PushUpdateMsg(msgId, Conf.Language(138), 3000)
+	util.PushMsg(Conf.Language(138), 3000)
 	time.Sleep(1 * time.Second)
 	if initErr := IndexRepo("[Init] Init data repo"); nil != initErr {
-		util.PushUpdateMsg(msgId, fmt.Sprintf(Conf.Language(140), initErr), 7000)
+		util.PushErrMsg(fmt.Sprintf(Conf.Language(140), initErr), 0)
 	}
 	return
 }