Updates xml error unmarshalling to look at all request id keys when p…

[cenedhryn]

…opulating error information

Motivation and Context

When XML type error responses are received and the chain AwsS3ProtocolFactory -> AwsXmlErrorTransformer -> AwsXmlErrorUnmarshaller is activated, responses with empty body only populate the requestId value in the exception if the header contains the key x-amzn-RequestId and not the alternate x-amz-request-id

Description

Attempts to find any valid request id key in the response headers

Testing

Validated using the S3 head object API call

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)

Checklist

• I have read the CONTRIBUTING document
• Local run of mvn install succeeds
• My code follows the code style of this project
• My change requires a change to the Javadoc documentation
• I have updated the Javadoc documentation accordingly
• I have read the README document
• I have added tests to cover my changes
• All new and existing tests passed
• A short description of the change has been added to the CHANGELOG
• My change is to implement 1.11 parity feature and I have updated LaunchChangelog

License

• I confirm that this pull request can be released under the Apache 2 license

[dagnir]

Code LGTM. Working on adding a test.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

100.0% Coverage
0.0% Duplication