Skip to content

Commit 85362aa

Browse files
author
Joshua Toth
authored
Merge pull request #1 from PageUpPeopleOrg/cannedaclfors3
Add a canned ACL for the S3 upload to that we don't have permission i…
2 parents fcd1d39 + 7c1577b commit 85362aa

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

backup.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,8 @@ module.exports = function(config, context, done) {
4646
s3.upload({
4747
Bucket: config.backup.bucket,
4848
Key: key,
49-
Body: data
49+
Body: data,
50+
ACL: 'bucket-owner-full-control'
5051
}, function(err) {
5152
if (err) return next(err);
5253
log('[segment %s] Uploaded dynamo backup to s3://%s/%s', index, config.backup.bucket, key);

s3-snapshot.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,8 @@ module.exports = function(config, done) {
4747
var upload = s3.upload({
4848
Bucket: config.destination.bucket,
4949
Key: config.destination.key,
50-
Body: gzip
50+
Body: gzip,
51+
ACL: 'bucket-owner-full-control'
5152
}).on('httpUploadProgress', function(details) {
5253
if (details.part !== partsLoaded) {
5354
log(

0 commit comments

Comments
 (0)