Since file is public via the ACL, is the Sid: PublicReadGetObject is superfluous.

Mike Kennedy

Answered 3 years ago

Sid is just an optional statement identifier. Think of it as a name, it is not imparting anything beyond a description. Perhaps a better statement description would have been more helpful. It’s just saying that this is a public file – "PublicReadGetObject". However, with the boolean condition we’re now saying that this public file should only be made available via SSL.

lincupel

Answered 3 years ago

Ditto what Mike said. Thanks Mike

Barry Sheward

Answered 3 years ago

I didn’t see anything in this video related to S3 ACLs. Like Mike said, the Sid value is irrelevant. The critical part of this whole video is the "Effect" : "Deny" with the condition: "Condition":{ "Bool": { "aws:SecureTransport": false }}