Sync with upstream

common/scripts/version.txt

@@ -1 +1 @@
-"0.7.422"
+"0.7.423"

foundations/core/packages/storage-client/src/__tests__/datalake-storage.test.ts

@@ -57,14 +57,24 @@ describe('DatalakeStorage', () => {
       expect(url).toBe(`${baseUrl}/blob/${workspace}/${file}`)
     })
 
-    it('should handle special characters in parameters', () => {
+    it('should encode special characters in parameters', () => {
       const workspace = 'test workspace'
       const file = 'file 123'
       const filename = 'my document.pdf'
 
       const url = storage.getFileUrl(workspace, file, filename)
 
-      expect(url).toBe(`${baseUrl}/blob/${workspace}/${file}/${filename}`)
+      expect(url).toBe(`${baseUrl}/blob/test%20workspace/file%20123/my%20document.pdf`)
+    })
+
+    it('should encode slash in filename as a single path segment', () => {
+      const workspace = 'test-workspace'
+      const file = 'file-123'
+      const filename = 'folder/document.pdf'
+
+      const url = storage.getFileUrl(workspace, file, filename)
+
+      expect(url).toBe(`${baseUrl}/blob/${workspace}/${file}/folder%2Fdocument.pdf`)
     })
 
     it('should handle base URL with trailing slash', () => {

foundations/core/packages/storage-client/src/__tests__/front-storage.test.ts

@@ -55,14 +55,24 @@ describe('FrontStorage', () => {
       expect(url).toBe(`${baseUrl}/${workspace}/${filename}?file=${file}&workspace=${workspace}`)
     })
 
-    it('should handle special characters in workspace and file names', () => {
+    it('should encode special characters in workspace, file, and filename', () => {
       const workspace = 'test workspace'
       const file = 'file 123'
       const filename = 'my document.pdf'
 
       const url = storage.getFileUrl(workspace, file, filename)
 
-      expect(url).toBe(`${baseUrl}/${workspace}/${filename}?file=${file}&workspace=${workspace}`)
+      expect(url).toBe(`${baseUrl}/test%20workspace/my%20document.pdf?file=file%20123&workspace=test%20workspace`)
+    })
+
+    it('should encode slash in filename as a single path segment', () => {
+      const workspace = 'test-workspace'
+      const file = 'file-123'
+      const filename = 'folder/document.pdf'
+
+      const url = storage.getFileUrl(workspace, file, filename)
+
+      expect(url).toBe(`${baseUrl}/${workspace}/folder%2Fdocument.pdf?file=${file}&workspace=${workspace}`)
     })
 
     it('should handle base URL with trailing slash', () => {

foundations/core/packages/storage-client/src/client/datalake.ts

@@ -16,6 +16,7 @@
 import { concatLink } from '@hcengineering/core'
 import { FileStorage, FileStorageUploadOptions } from '../types'
 import { uploadMultipart, uploadXhr } from '../upload'
+import { encodePathSegment } from './utils'
 
 const getPathname = (url: string): string => {
   const base = window?.location?.href !== undefined ? window.location.href : 'http://localhost'
@@ -27,7 +28,12 @@ export class DatalakeStorage implements FileStorage {
   constructor (private readonly baseUrl: string) {}
 
   getFileUrl (workspace: string, file: string, filename?: string): string {
-    const path = filename !== undefined ? `/blob/${workspace}/${file}/${filename}` : `/blob/${workspace}/${file}`
+    const encodedWorkspace = encodePathSegment(workspace)
+    const encodedFile = encodePathSegment(file)
+    const path =
+      filename !== undefined
+        ? `/blob/${encodedWorkspace}/${encodedFile}/${encodePathSegment(filename)}`
+        : `/blob/${encodedWorkspace}/${encodedFile}`
     return concatLink(this.baseUrl, path)
   }