SQS Message fix for eventSourceArn for JSON deserialization.#103
SQS Message fix for eventSourceArn for JSON deserialization.#103srayhunter wants to merge 1 commit intoaws:masterfrom srayhunter:bugfix/sqs-message_eventSourceARN
Conversation
to eventSourceARN for json deserialization.
|
Thanks for the change. I'd like to understand what issue you faced that this change fixes? I ask because the runtime's (de)serializer is configured to expect the key Using and this payload: I get back: |
|
@mukhtar what are you using to parse the json message? If using gson, there are issue with case. For example in SQSEvent there is the following |
|
Hi @srayhunter, The Lambda service receives json events which sometimes don't meet the normal Java naming conventions. To handle this custom serialization is done on the customers behalf. You can find the implementation in the serialization module. In order to unit test Lambda function handlers we advise that you use the Testing module in order to have this custom serialization exercised. |
Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
Issue #, if available:
Description of changes:
Fixing naming issue with sqs message eventSourceArn needs to be renamed to eventSourceARN for json deserialization.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.