From 3785cc06e6554f4e5d0d93304e3d7fb9764b5018 Mon Sep 17 00:00:00 2001 From: zhom <2717306+zhom@users.noreply.github.com> Date: Sat, 3 Jan 2026 21:42:24 +0400 Subject: [PATCH] refactor: handle BucketAlreadyOwnedByYou error --- donut-sync/src/sync/sync.service.ts | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/donut-sync/src/sync/sync.service.ts b/donut-sync/src/sync/sync.service.ts index 243479f..e64731b 100644 --- a/donut-sync/src/sync/sync.service.ts +++ b/donut-sync/src/sync/sync.service.ts @@ -90,9 +90,19 @@ export class SyncService implements OnModuleInit { new CreateBucketCommand({ Bucket: this.bucket }), ); this.s3Ready = true; - } catch (createError) { - console.error("Failed to create S3 bucket:", createError); - throw createError; + } catch (createError: unknown) { + // BucketAlreadyOwnedByYou means the bucket exists and we own it - this is fine + const isAlreadyOwned = + createError && + typeof createError === "object" && + "name" in createError && + createError.name === "BucketAlreadyOwnedByYou"; + if (isAlreadyOwned) { + this.s3Ready = true; + } else { + console.error("Failed to create S3 bucket:", createError); + throw createError; + } } } else { console.error("S3 connection failed:", error);