Nugine / s3s

S3 Service Adapter
Apache License 2.0
133 stars 33 forks source link

Fix s3UnwrappedXmlOutput of LocationConstraint #127

Closed nomick closed 8 months ago

nomick commented 8 months ago

The GetBucketLocation operation differs from other operations as it does not double the LocationConstraint XML tag. See https://smithy.io/2.0/aws/customizations/s3-customizations.html for details. Note that the example at https://docs.aws.amazon.com/AmazonS3/latest/API/API_GetBucketLocation.html is also wrong, as the AWS-Client will also not accept it.

This patch fixes the output form

<LocationConstraint>
   <LocationConstraint>string</LocationConstraint>
</LocationConstraint>

to

<LocationConstraint>string</LocationConstraint>

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.