Aws-lambda-dotnet: Prise en charge de System.Text.Json pour le SDK AWS Lambda ?

Dans le cadre de la migration de certains services de travail .NET Core 3.0 existants exécutés sur EC2 vers AWS Lambda, nous avons codé une implémentation personnalisée ILambdaSerializer pour gérer les charges utiles de désérialisation vers la fonction Lambda, telles que SQSEvent .

Bien que cela soit relativement simple à faire, il serait bien qu'une implémentation de sérialiseur pour System.Text.Json soit intégrée au SDK Lambda pour éviter d'avoir à copier-coller sur différentes bases de code de fonction.

L'un des inconvénients de l'état actuel des choses est que PropertyNameCaseInsensitive doit être défini sur true dans les options du sérialiseur, car les événements n'utilisent pas une dénomination cohérente PascalCase par rapport à camelCase, ce qui encourt une pénalité de performance (je n'ai pas de quantificateur pour combien la pénalité est). Dans le cas de SQSEvent , je crois que c'est eventSourceARN qui le déclenche car il ne correspond pas à EventSourceArn .

J'imagine que cela pourrait être facilement corrigé dans les bibliothèques d'événements en ajoutant des annotations d'attribut [JsonPropertyName("...")] aux propriétés des modèles.

Je pense que d'autres avantages pourraient être obtenus s'il y avait des méthodes asynchrones sur l'interface pour tirer parti du support asynchrone de System.Text.Json.JsonSerializer et consommer directement les Stream , mais c'est un changement plus important qui fournit simplement une implémentation basée sur l'interface actuelle.

Ci-dessous le code que nous utilisons. Il pourrait bénéficier d'une optimisation supplémentaire, mais tel qu'il est actuellement, il fonctionne bien pour nous :

using System.IO;
using System.Text.Json;
using Amazon.Lambda.Core;

namespace MyLambdaFunction
{
    public sealed class SystemTextJsonLambdaSerializer : ILambdaSerializer
    {
        private readonly JsonSerializerOptions _options;

        public SystemTextJsonLambdaSerializer()
        {
            // SQSEvent does not use a consistent camel/Pascal case naming convention,
            // so as we cannot annotate with attributes, the options need to be case
            // insensitive to be tolerant for things to work properly when deserializing.
            _options = new JsonSerializerOptions()
            {
                IgnoreNullValues = true,
                PropertyNameCaseInsensitive = true,
            };
        }

        public T Deserialize<T>(Stream requestStream)
        {
            using var copy = new MemoryStream();
            requestStream.CopyTo(copy);

            byte[] utf8Json = copy.ToArray();

            return JsonSerializer.Deserialize<T>(utf8Json, _options);
        }

        public void Serialize<T>(T response, Stream responseStream)
        {
            using var writer = new Utf8JsonWriter(responseStream);
            JsonSerializer.Serialize(writer, response, _options);
        }
    }
}

/cc @normj