fix (share): filename issue in shared link

mickael-kerjean · mickael-kerjean · commit 8b0c5e590939 · 2024-12-10T21:54:42.000+11:00
diff --git a/public/assets/pages/ctrl_sharepage.js b/public/assets/pages/ctrl_sharepage.js
@@ -26,7 +26,7 @@ export default function(render) {
         else if (step === "code") return ctrlEmailCodeVerification(render, { shareID, setState });
         else if (step === "done") {
             if (isDir(state["path"])) navigate(toHref(`/files/?share=${shareID}`));
-            else navigate(toHref(`/view/${basename(state["path"])}?share=${shareID}&nav=false`));
+            else navigate(toHref(`/view/${encodeURIComponent(basename(state["path"]))}?share=${shareID}&nav=false`));
             return rxjs.EMPTY;
         }
         else assert.fail(`unknown step: "${step}"`);

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ export default function(render) {`
`26`	`26`	`else if (step === "code") return ctrlEmailCodeVerification(render, { shareID, setState });`
`27`	`27`	`else if (step === "done") {`
`28`	`28`	if (isDir(state["path"])) navigate(toHref(`/files/?share=${shareID}`));
`29`		- else navigate(toHref(`/view/${basename(state["path"])}?share=${shareID}&nav=false`));
	`29`	+ else navigate(toHref(`/view/${encodeURIComponent(basename(state["path"]))}?share=${shareID}&nav=false`));
`30`	`30`	`return rxjs.EMPTY;`
`31`	`31`	`}`
`32`	`32`	else assert.fail(`unknown step: "${step}"`);