Update SQSRequestHandler.cs - Fix 2811

[indy-singh]

This quick fixes dynamically pulling out a property see #2811

[CLAassistant]

All committers have signed the CLA.

[nr-ahemsath]

Thank you for submitting a PR. At first glance, your changes make sense. However, on our end we need to do some more research to understand how this scenario can occur, and come up with a reliable reproduction case. Thank your for your patience while we do the analysis.

[indy-singh]

Hi @nr-ahemsath,

It will be an empty ReceiveMessageResponse, so the scenario is the queue was polled but it had no messages to action. So an empty response T was received. Depending on how the consumer has configured their AWS SDK; the collection on T is either empty or null.

https://docs.aws.amazon.com/AWSSimpleQueueService/latest/APIReference/API_ReceiveMessage.html#API_ReceiveMessage_ResponseElements

https://github.com/aws/aws-sdk-net/blob/b5c10d5d48299f6c68286ee48039bc4ecbc21019/sdk/src/Services/SQS/Generated/Model/ReceiveMessageResponse.cs#L37

Happy to close the PR, if you guys want to deep dive on it.

Cheers,
Indy

[nr-ahemsath]

@indy-singh Thanks for the additional info, that will be very helpful. We're happy to leave the PR open for now.

[nr-ahemsath]

@indy-singh Thanks again for the initial take on this fix. This PR has a more complete version of the fix, plus testing: #2833

Will close this one now.