fix:Return the backend URL for private files of local file provider

sradevski · sradevski · commit b22d407a94a9 · 2024-07-29T20:15:57.000+03:00
diff --git a/packages/modules/providers/file-local/src/services/local-file.ts b/packages/modules/providers/file-local/src/services/local-file.ts
@@ -77,7 +77,6 @@ export class LocalFileService extends AbstractFileProviderService {
     return
   }
 
-  // For private files, we simply return the file path, which can then be loaded manually by the backend.
   // The local file provider doesn't support presigned URLs for private files (i.e files not placed in /static).
   async getPresignedDownloadUrl(
     file: FileTypes.ProviderGetFileDTO
@@ -96,10 +95,6 @@ export class LocalFileService extends AbstractFileProviderService {
       )
     }
 
-    if (isPrivate) {
-      return filePath
-    }
-
     return this.getUploadFileUrl(file.fileKey)
   }
 

Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,6 @@ export class LocalFileService extends AbstractFileProviderService {`
`77`	`77`	`return`
`78`	`78`	`}`
`79`	`79`
`80`		`- // For private files, we simply return the file path, which can then be loaded manually by the backend.`
`81`	`80`	`// The local file provider doesn't support presigned URLs for private files (i.e files not placed in /static).`
`82`	`81`	`async getPresignedDownloadUrl(`
`83`	`82`	`file: FileTypes.ProviderGetFileDTO`
`@@ -96,10 +95,6 @@ export class LocalFileService extends AbstractFileProviderService {`
`96`	`95`	`)`
`97`	`96`	`}`
`98`	`97`
`99`		`- if (isPrivate) {`
`100`		`- return filePath`
`101`		`- }`
`102`		`-`
`103`	`98`	`return this.getUploadFileUrl(file.fileKey)`
`104`	`99`	`}`
`105`	`100`