Merge pull request #108 from glennpratt/CPD-885-s3-acl

CPD-885 Grant access to S3 artifacts created in other accounts.
acquia · Jul 7, 2016 · 916dc42 · 916dc42
2 parents 1397919 + fc3b144
commit 916dc42
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/lib/moonshot/artifact_repository/s3_bucket.rb b/lib/moonshot/artifact_repository/s3_bucket.rb
@@ -40,6 +40,7 @@ def filename_for_version(version_name)
 
   def upload_to_s3(file, key)
     s3_client.put_object(
+      acl: 'bucket-owner-full-control',
       key: key,
       body: File.open(file),
       bucket: @bucket_name,

diff --git a/moonshot.gemspec b/moonshot.gemspec
@@ -17,6 +17,9 @@ Gem::Specification.new do |s|
   s.add_dependency('interactive-logger', '~> 0.1.1')
   s.add_dependency('rotp', '~> 2.1.1')
   s.add_dependency('ruby-duration', '~> 3.2.3')
+  # Pin back activesupport (ruby-duration dependency) until we only support
+  # Ruby >= 2.2.2.
+  s.add_dependency('activesupport', '< 5.0.0')
   s.add_dependency('thor', '~> 0.19.1')
   s.add_dependency('semantic')
   s.add_dependency('vandamme')