Fixed the JSON deserialization error in Cognito triggered lambda - Issue #1644

[ankushjain358]

Issue #1644

Issue #, if available:

Description of changes: Made the required properties nullable

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[ashovlin]

Hey @ankushjain358 - the change looks good, but could you add unit test that exercise null values? Reading #1644 (comment), I'm not too concerned about verifying correctness now, but would rather guard against future regressions. We've long mused about someday generating the event models as opposed to handwriting them, and wouldn't want to flip the nullability back accidentally if we do (and like @ashishdhingra commented in the issue, the documentation isn't always clear about nullability).

Also, if you do revisit the change, could you take this to 3.0.0? We'll handle the changelog as part of the release process.

aws-lambda-dotnet/Libraries/src/Amazon.Lambda.CognitoEvents/Amazon.Lambda.CognitoEvents.csproj

Line 9 in d2b507c

<VersionPrefix>2.2.0</VersionPrefix>

[ankushjain358]

@ashovlin - I have pushed the requested changes. Please take a look.

[normj]

Thanks for the PR. Version 3.0.0 has just been released.