Refactor/cloud breach s3

[andrew-aiken]

Overview of Changes

• Updated deprecated code (https://github.com/RhinoSecurityLabs/cloudgoat/pull/201)
• Formatted code terraform fmt
• Linting
  • Adding variable types
• Remove duplicated & redundant code
  • ACL block by default (https://github.com/RhinoSecurityLabs/cloudgoat/issues/198)
  • Similar change to (https://github.com/RhinoSecurityLabs/cloudgoat/pull/193)
• PR template to repo

Testing

Tested with version 1.2.0 & 1.5.6 of Terraform