S3 get_bucket_location Would Benefit from Deprecation Attribute

[marksieklucki]

Describe the feature

The AWS S3 API documentation notes that HEAD should be used instead of get_bucket_location, presumably because of the non-standard return values for us-east-1/eu-west-1.

Currently, the rust SDK doesn't include a deprecated attribute, which makes it easy for developers to use a function call that is easy to misuse.

Use Case

Deprecation attributes on get_bucket_location would help developers avoid using a method that is easy to misuse (in favor of the recommended HEAD method).

Proposed Solution

No response

Other Information

No response

Acknowledgements

• I may be able to implement this feature request
• This feature might incur a breaking change

A note for the community

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue, please leave a comment

[ysaito1001]

Thank you for the suggestion. While it is mentioned in the documentation, it's clearer to users if the message shows up at compile time. We can consider handling this during code generation for S3 by special-casing get_bucket_location to include the deprecated attribute.