hashicorp / terraform-provider-aws

The AWS Provider enables Terraform to manage AWS resources.
https://registry.terraform.io/providers/hashicorp/aws
Mozilla Public License 2.0
9.76k stars 9.11k forks source link

[Docs]: S3 Object Lock no longer needs a token field #36938

Closed tanelso2 closed 2 months ago

tanelso2 commented 5 months ago

Documentation Link

https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_bucket_object_lock_configuration#token

Description

Hi there,

The aws_s3_bucket_object_lock_configuration object type has a field called token that it claims is needed to enable Object Lock on an existing S3 bucket.

This is no longer the case. As of November 2023, AWS allows you to enable Object Lock without specifying a token, and the current versions of the Object Lock documentation makes no reference to the token field.

I just tested this out myself with a quick demo. I was able to add Object Lock to an existing bucket without a token.

We should update the documentation to make it clear that the token is deprecated and no longer needed. The current documentation implies the opposite.

References

https://docs.aws.amazon.com/AmazonS3/latest/userguide/object-lock-configure.html https://aws.amazon.com/about-aws/whats-new/2023/11/amazon-s3-enabling-object-lock-buckets/

Relates https://github.com/hashicorp/terraform-provider-aws/pull/36530.

Would you like to implement a fix?

Yes

github-actions[bot] commented 5 months ago

Community Note

Voting for Prioritization

Volunteering to Work on This Issue

Jyots6914 commented 3 months ago

I will work on it and I will raise PR

github-actions[bot] commented 2 months ago

[!WARNING] This issue has been closed, meaning that any additional comments are hard for our team to see. Please assume that the maintainers will not see them.

Ongoing conversations amongst community members are welcome, however, the issue will be locked after 30 days. Moving conversations to another venue, such as the AWS Provider forum, is recommended. If you have additional concerns, please open a new issue, referencing this one where needed.

github-actions[bot] commented 2 months ago

This functionality has been released in v5.58.0 of the Terraform AWS Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

tanelso2 commented 2 months ago

Fantastic, thanks everyone who worked on this!

github-actions[bot] commented 1 month ago

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.