fix: check if image exists before setting it (#636)

Co-authored-by: orig <[email protected]>
origranot · Dec 24, 2023 · 72e1083 · 72e1083
1 parent 78a8bad
commit 72e1083
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/apps/backend/src/auth/providers/providers.controller.ts b/apps/backend/src/auth/providers/providers.controller.ts
@@ -46,7 +46,10 @@ export class ProvidersController {
     }
 
     if (req.user.picture && this.configService.getConfig().storage.enable) {
-      await this.storageService.uploadImageFromUrl(req.user.picture, `${PROFILE_PICTURE_PREFIX}/${user.id}`);
+      const imagePath = `${PROFILE_PICTURE_PREFIX}/${user.id}`;
+      if (!this.storageService.exists(imagePath)) {
+        await this.storageService.uploadImageFromUrl(req.user.picture, imagePath);
+      }
     }
 
     const domain = this.configService.getConfig().front.domain;

diff --git a/apps/backend/src/storage/storage.service.ts b/apps/backend/src/storage/storage.service.ts
@@ -56,6 +56,20 @@ export class StorageService {
     const data = await this.s3Client.deleteObject(params);
     return data;
   }
+
+  async exists(name: string) {
+    const params: PutObjectCommandInput = {
+      Bucket: this.config.getConfig().storage.bucket,
+      Key: name,
+    };
+
+    try {
+      await this.s3Client.headObject(params);
+      return true;
+    } catch (error) {
+      return false;
+    }
+  }
 }
 
 export const PROFILE_PICTURE_PREFIX = 'profile-pictures';