Using an existing S3 bucket blows up backups about backup

[joris]

This patch would fix it:

diff --git a/lib/backup/connection/s3.rb b/lib/backup/connection/s3.rb
index fea683c..743c20f 100644
--- a/lib/backup/connection/s3.rb
+++ b/lib/backup/connection/s3.rb
@@ -37,8 +37,14 @@ module Backup
 
       # Wrapper for the Bucket object
       def bucket
-        bucket = service.buckets.build(s3_bucket)
-        bucket.save
+        begin
+          # Find existing bucket:
+          bucket = service.buckets.find(s3_bucket)
+        rescue ::S3::Error::NoSuchBucket => e
+          # Apparently the bucket doesn't exist yet, so create a new one:
+          bucket = service.buckets.build(s3_bucket)
+          bucket.save
+        end
         bucket.retrieve
       end

[mrrooijen]

Thanks! Patching now.. one sec!

[mrrooijen]

Patch: 5fc38f1

[mrrooijen]

Released Backup v2.3.3.1 [Patch] that fixes this.