Merge pull request #100 from peavers/master

lukemelia · web-flow · commit 71a84ad4735e · 2019-02-14T16:41:33.000-05:00
Updated bucket policy based on latest AWS recommendations
diff --git a/README.md b/README.md
@@ -285,16 +285,14 @@ If you want the contents of the S3 bucket to be accessible to the world, the fol
 
 ```js
 {
+    "Version": "2012-10-17",
     "Statement": [
         {
-            "Sid": "Stmt1EmberCLIS3AccessPolicy",
+            "Sid": "PublicReadForGetBucketObjects",
             "Effect": "Allow",
-            "Action": [
-                "s3:GetObject"
-            ],
-            "Resource": [
-                "arn:aws:s3:::your-s3-bucket-name/*"
-            ]
+            "Principal": "*",
+            "Action": "s3:GetObject",
+            "Resource": "arn:aws:s3:::bucket-name/*"
         }
     ]
 }

Original file line number	Diff line number	Diff line change
`@@ -285,16 +285,14 @@ If you want the contents of the S3 bucket to be accessible to the world, the fol`
`285`	`285`
`286`	`286`	```js
`287`	`287`	`{`
	`288`	`+ "Version": "2012-10-17",`
`288`	`289`	`"Statement": [`
`289`	`290`	`{`
`290`		`- "Sid": "Stmt1EmberCLIS3AccessPolicy",`
	`291`	`+ "Sid": "PublicReadForGetBucketObjects",`
`291`	`292`	`"Effect": "Allow",`
`292`		`- "Action": [`
`293`		`- "s3:GetObject"`
`294`		`- ],`
`295`		`- "Resource": [`
`296`		`- "arn:aws:s3:::your-s3-bucket-name/*"`
`297`		`- ]`
	`293`	`+ "Principal": "*",`
	`294`	`+ "Action": "s3:GetObject",`
	`295`	`+ "Resource": "arn:aws:s3:::bucket-name/*"`
`298`	`296`	`}`
`299`	`297`	`]`
`300`	`298`	`}`