Fixed Deprecated aws_s3_bucket Options

[afrazkhan]

trafficstars

The following options were deprecated, and have either been updated, or replaced with the new necessary resources:

• override_json replaced with override_policy_documents option
• server_side_encryption_configuration replaced with aws_s3_bucket_server_side_encryption_configuration resource
• versioning replaced with aws_s3_bucket_versioning resource
• cors_rule replaced with aws_s3_bucket_cors_configuration resource
• acl replaced with aws_s3_bucket_acl resource

[bshabae]

The following options were deprecated, and have either been updated, or replaced with the new necessary resources:

• override_json replaced with override_policy_documents option
• server_side_encryption_configuration replaced with aws_s3_bucket_server_side_encryption_configuration resource
• versioning replaced with aws_s3_bucket_versioning resource
• cors_rule replaced with aws_s3_bucket_cors_configuration resource
• acl replaced with aws_s3_bucket_acl resource

Hi guys @dotCipher and @joe-niland!

Any updates on this? We are still seeing the following deprecation warning:

│ Warning: Argument is deprecated
│   on .terraform/modules/cloudfront_s3_site/main.tf line 248, in resource "aws_s3_bucket" "origin":
│  248: resource "aws_s3_bucket" "origin" {
│ 
│ Use the aws_s3_bucket_cors_configuration resource instead

[davidjeddy]

Bump, checking in on the progress of this patch.

[bshabae]

Bump, checking in on the progress of this patch.

Thank you! We are looking forward to this update.

[joe-niland]

Thanks @afrazkhan and sorry for the delay in reviewing.

There are some formatting issues. Could you please run the following to see the output?

make precommit/terraform

I believe they can be fixed by running terraform fmt -recursive

[mergify[bot]]

Thanks @afrazkhan for creating this pull request!

A maintainer will review your changes shortly. Please don't be discouraged if it takes a while.

While you wait, make sure to review our contributor guidelines.

[!TIP]

Need help or want to ask for a PR review to be expedited?

Join us on Slack in the #pr-reviews channel.

[mergify[bot]]

This pull request now has conflicts. Could you fix it @afrazkhan? 🙏

[mergify[bot]]

This PR has been closed due to inactivity and merge conflicts. Please resolve the conflicts and reopen if necessary.