Merge pull request #59033 from nextcloud/fix/files-external-issues

fix(files_external): properly handle API errors

Author: susnux

apps/files_external/src/components/AddExternalStorageDialog/AddExternalStorageDialog.vue

@@ -33,7 +33,7 @@ const open = defineModel<boolean>('open', { default: true })
 const {
 	storage = { backendOptions: {}, mountOptions: {}, type: isAdmin ? 'system' : 'personal' },
 } = defineProps<{
-	storage?: Partial<IStorage> & { backendOptions: IStorage['backendOptions'] }
+	storage?: Partial<IStorage>
 }>()
 
 defineEmits<{
@@ -88,7 +88,7 @@ watch(authMechanisms, () => {
 			:label="t('files_external', 'Folder name')"
 			required />
 
-		<MountOptions v-model="internalStorage.mountOptions" />
+		<MountOptions v-model="internalStorage.mountOptions!" />
 
 		<ApplicableEntities
 			v-if="isAdmin"
@@ -112,13 +112,13 @@ watch(authMechanisms, () => {
 			required />
 
 		<BackendConfiguration
-			v-if="backend"
+			v-if="backend && internalStorage.backendOptions"
 			v-model="internalStorage.backendOptions"
 			:class="$style.externalStorageDialog__configuration"
 			:configuration="backend.configuration" />
 
 		<AuthMechanismConfiguration
-			v-if="authMechanism"
+			v-if="authMechanism && internalStorage.backendOptions"
 			v-model="internalStorage.backendOptions"
 			:class="$style.externalStorageDialog__configuration"
 			:authMechanism="authMechanism" />

apps/files_external/src/components/AddExternalStorageDialog/MountOptions.vue

@@ -14,17 +14,14 @@ import NcButton from '@nextcloud/vue/components/NcButton'
 import NcCheckboxRadioSwitch from '@nextcloud/vue/components/NcCheckboxRadioSwitch'
 import NcIconSvgWrapper from '@nextcloud/vue/components/NcIconSvgWrapper'
 import NcSelect from '@nextcloud/vue/components/NcSelect'
+import { parseMountOptions } from '../../store/storages.ts'
 import { MountOptionsCheckFilesystem } from '../../types.ts'
 
 const mountOptions = defineModel<Partial<IMountOptions>>({ required: true })
 watchEffect(() => {
 	if (Object.keys(mountOptions.value).length === 0) {
-		mountOptions.value.encrypt = true
-		mountOptions.value.previews = true
-		mountOptions.value.enable_sharing = false
-		mountOptions.value.filesystem_check_changes = MountOptionsCheckFilesystem.OncePerRequest
-		mountOptions.value.encoding_compatibility = false
-		mountOptions.value.readonly = false
+		// parse and initialize with defaults if needed
+		mountOptions.value = parseMountOptions(mountOptions.value)
 	}
 })
 

apps/files_external/src/store/storages.ts

@@ -3,14 +3,15 @@
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
 
-import type { IStorage } from '../types.d.ts'
+import type { IStorage } from '../types.ts'
 
 import axios from '@nextcloud/axios'
 import { loadState } from '@nextcloud/initial-state'
 import { addPasswordConfirmationInterceptors, PwdConfirmationMode } from '@nextcloud/password-confirmation'
 import { generateUrl } from '@nextcloud/router'
 import { defineStore } from 'pinia'
 import { ref, toRaw } from 'vue'
+import { MountOptionsCheckFilesystem } from '../types.ts'
 
 const { isAdmin } = loadState<{ isAdmin: boolean }>('files_external', 'settings')
 
@@ -30,7 +31,7 @@ export const useStorages = defineStore('files_external--storages', () => {
 			toRaw(storage),
 			{ confirmPassword: PwdConfirmationMode.Strict },
 		)
-		globalStorages.value.push(data)
+		globalStorages.value.push(parseStorage(data))
 	}
 
 	/**
@@ -45,7 +46,7 @@ export const useStorages = defineStore('files_external--storages', () => {
 			toRaw(storage),
 			{ confirmPassword: PwdConfirmationMode.Strict },
 		)
-		userStorages.value.push(data)
+		userStorages.value.push(parseStorage(data))
 	}
 
 	/**
@@ -77,7 +78,7 @@ export const useStorages = defineStore('files_external--storages', () => {
 			{ confirmPassword: PwdConfirmationMode.Strict },
 		)
 
-		overrideStorage(data)
+		overrideStorage(parseStorage(data))
 	}
 
 	/**
@@ -87,7 +88,7 @@ export const useStorages = defineStore('files_external--storages', () => {
 	 */
 	async function reloadStorage(storage: IStorage) {
 		const { data } = await axios.get(getUrl(storage))
-		overrideStorage(data)
+		overrideStorage(parseStorage(data))
 	}
 
 	// initialize the store
@@ -111,6 +112,7 @@ export const useStorages = defineStore('files_external--storages', () => {
 		const url = `apps/files_external/${type}`
 		const { data } = await axios.get<Record<number, IStorage>>(generateUrl(url))
 		return Object.values(data)
+			.map(parseStorage)
 	}
 
 	/**
@@ -150,3 +152,45 @@ export const useStorages = defineStore('files_external--storages', () => {
 		}
 	}
 })
+
+/**
+ * @param storage - The storage from API
+ */
+function parseStorage(storage: IStorage) {
+	return {
+		...storage,
+		mountOptions: parseMountOptions(storage.mountOptions),
+	}
+}
+
+/**
+ * Parse the mount options and convert string boolean values to
+ * actual booleans and numeric strings to numbers
+ *
+ * @param options - The mount options to parse
+ */
+export function parseMountOptions(options: IStorage['mountOptions']) {
+	const mountOptions = { ...options }
+	mountOptions.encrypt = convertBooleanOptions(mountOptions.encrypt, true)
+	mountOptions.previews = convertBooleanOptions(mountOptions.previews, true)
+	mountOptions.enable_sharing = convertBooleanOptions(mountOptions.enable_sharing, false)
+	mountOptions.filesystem_check_changes = typeof mountOptions.filesystem_check_changes === 'string'
+		? Number.parseInt(mountOptions.filesystem_check_changes)
+		: (mountOptions.filesystem_check_changes ?? MountOptionsCheckFilesystem.OncePerRequest)
+	mountOptions.encoding_compatibility = convertBooleanOptions(mountOptions.encoding_compatibility, false)
+	mountOptions.readonly = convertBooleanOptions(mountOptions.readonly, false)
+	return mountOptions
+}
+
+/**
+ * Convert backend encoding of boolean options
+ *
+ * @param option - The option value from API
+ * @param fallback - The fallback (default) value
+ */
+function convertBooleanOptions(option: unknown, fallback = false) {
+	if (option === undefined) {
+		return fallback
+	}
+	return option === true || option === 'true' || option === '1'
+}

apps/files_external/src/views/ExternalStoragesSection.vue

@@ -59,7 +59,8 @@ async function addStorage(storage?: Partial<IStorage>) {
 		}
 		newStorage.value = undefined
 	} catch (error) {
-		logger.error('Failed to add external storage', { error })
+		logger.error('Failed to add external storage', { error, storage })
+		newStorage.value = { ...storage }
 		showDialog.value = true
 	}
 }
@@ -134,8 +135,8 @@ async function addStorage(storage?: Partial<IStorage>) {
 		</NcButton>
 
 		<AddExternalStorageDialog
-			v-model="newStorage"
 			v-model:open="showDialog"
+			:storage="newStorage"
 			@close="addStorage" />
 
 		<UserMountSettings v-if="settings.isAdmin" />

dist/AuthMechanismRsa-CX_mdruD.chunk.mjs

@@ -0,0 +1,2 @@
+import{f as g,B as y,u as o,o as n,g as p,N as h,w as v,v as _,t as V,E as k,i as x,c as d,H as K,I as M,K as w,L as U,h as f,r as q}from"./runtime-dom.esm-bundler-CKxgtjB6.chunk.mjs";import{c as E}from"./index-2a6kOrDy.chunk.mjs";import{a as L}from"./index-C1xmmKTZ-DF4bDPEh.chunk.mjs";import{t as s}from"./translation-DoG5ZELJ-XUrtIRvk.chunk.mjs";import{g as N}from"./createElementId-DhjFt1I9-DdwCqgaq.chunk.mjs";import{N as S}from"./autolink-U5pBzLgI-D1NMH9bI.chunk.mjs";import{N as j}from"./NcSelect-B1uITk_3-0e2p8qEi.chunk.mjs";import{N as A}from"./NcCheckboxRadioSwitch-D0gFwEVl-BbcIGLyw.chunk.mjs";import{N as B}from"./NcPasswordField-BOLzDHBJ-CKSnPt2n.chunk.mjs";import{_ as C}from"./TrashCanOutline-DmYYOQ4b.chunk.mjs";import{C as c,a as b}from"./types-Dild8a3G.chunk.mjs";import{l as z}from"./logger-resIultJ.chunk.mjs";const P=g({__name:"ConfigurationEntry",props:k({configKey:{},configOption:{}},{modelValue:{type:[String,Boolean],default:""},modelModifiers:{}}),emits:["update:modelValue"],setup(e){const a=y(e,"modelValue");return(t,i)=>e.configOption.type!==o(c).Boolean?(n(),p(h(e.configOption.type===o(c).Password?o(B):o(C)),{key:0,modelValue:a.value,"onUpdate:modelValue":i[0]||(i[0]=l=>a.value=l),name:e.configKey,required:!(e.configOption.flags&o(b).Optional),label:e.configOption.value,title:e.configOption.tooltip},null,8,["modelValue","name","required","label","title"])):(n(),p(o(A),{key:1,modelValue:a.value,"onUpdate:modelValue":i[1]||(i[1]=l=>a.value=l),type:"switch",title:e.configOption.tooltip},{default:v(()=>[_(V(e.configOption.value),1)]),_:1},8,["modelValue","title"]))}}),R=g({__name:"AuthMechanismRsa",props:k({authMechanism:{}},{modelValue:{required:!0},modelModifiers:{}}),emits:["update:modelValue"],setup(e){const a=y(e,"modelValue"),t=q();x(t,()=>{t.value&&(a.value.private_key="",a.value.public_key="")});async function i(){try{const{data:l}=await E.post(N("/apps/files_external/ajax/public_key.php"),{keyLength:t.value});a.value.private_key=l.data.private_key,a.value.public_key=l.data.public_key}catch(l){z.error("Error generating RSA key pair",{error:l}),L(s("files_external","Error generating key pair"))}}return(l,m)=>(n(),d("div",null,[(n(!0),d(K,null,M(e.authMechanism.configuration,(r,u)=>w((n(),p(P,{key:r.value,modelValue:a.value[u],"onUpdate:modelValue":O=>a.value[u]=O,configKey:u,configOption:r},null,8,["modelValue","onUpdate:modelValue","configKey","configOption"])),[[U,!(r.flags&o(b).Hidden)]])),128)),f(o(j),{modelValue:t.value,"onUpdate:modelValue":m[0]||(m[0]=r=>t.value=r),clearable:!1,inputLabel:o(s)("files_external","Key size"),options:[1024,2048,4096],required:""},null,8,["modelValue","inputLabel"]),f(o(S),{disabled:!t.value,wide:"",onClick:i},{default:v(()=>[_(V(o(s)("files_external","Generate keys")),1)]),_:1},8,["disabled"])]))}}),$=Object.freeze(Object.defineProperty({__proto__:null,default:R},Symbol.toStringTag,{value:"Module"}));export{$ as A,P as _};
+//# sourceMappingURL=AuthMechanismRsa-CX_mdruD.chunk.mjs.map