fix: Correctly handle null values while deserializing JSON to MemoryStream in Amazon.Lambda.Serialization.Json

Author: 96malhar

Libraries/src/Amazon.Lambda.Serialization.Json/JsonToMemoryStreamDataConverter.cs

@@ -24,6 +24,9 @@ public override bool CanConvert(Type objectType)
         public override object ReadJson(JsonReader reader, Type objectType, object existingValue, NewtonsoftJsonSerializer serializer)
         {
             var dataBase64 = reader.Value as string;
+            if (dataBase64 is null)
+                return null;
+
             return Common.Base64ToMemoryStream(dataBase64);
         }
 

Libraries/test/EventsTests.Shared/EventTests.cs

@@ -1302,6 +1302,7 @@ public void SQSTest(Type serializerType)
                     Assert.Equal("b2", attribute2.StringListValues[1]);
 
                     Assert.Equal("String", attribute2.DataType);
+                    Assert.Null(attribute2.BinaryValue);
                 }
 
                 Handle(sqsEvent);

Libraries/test/EventsTests.Shared/sqs-event.json

@@ -26,8 +26,9 @@
         "Attribute2": {
           "stringValue": "AttributeValue2",
           "stringListValues": [ "b1", "b2" ],
-          "binaryListValues": [  ],
-          "dataType": "String"
+          "binaryListValues": [],
+          "dataType": "String",
+          "binaryValue": null
         }
       }
     }