Admiral-Piett / goaws

AWS (SQS/SNS) Clone for Development testing
MIT License
778 stars 145 forks source link

Move ErrorResponse RequestId out of the Error element #199

Closed andygrundman closed 5 years ago

andygrundman commented 5 years ago

While working with an overly strict client library, I ran across a minor bug in goaws's ErrorResponse. According to the docs (link below), the RequestId element should be a sibling of Error, but goaws puts it inside. This patch should fix it. I apologize for not adding any tests, my Go knowledge is a bit too limited for that.

https://docs.aws.amazon.com/AWSSimpleQueueService/latest/SQSDeveloperGuide/sqs-api-responses.html