-
Notifications
You must be signed in to change notification settings - Fork 248
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Supply serde (de)-serializable input structs #382
Comments
I want to clarify here—the specific need you have is for lambda events, right? Not generalized for all structs necessarily? One thing you could do as a stopgap is use serde remote derive to derive serde onto the cloudwatch structs with the appropriate renames. |
Yes, the need is specifically for the embedded API call via CloudTrail CloudWatch Lambda event where the I will have a look at the remote derive functionality - I didn't know about it. Thanks for pointing that out. :) edit: well, since all models are non-exhaustive, I would have to implement every deserialization step manually using the builder pattern interface. |
closed, dupe of #269 |
|
Prerequisites
Question Description
https://github.com/LegNeato/aws-lambda-events currently offers structs for various Lambda inputs, but falls short when it comes to specific CloudWatch Events events, namely, API Calls via CloudTrail (for reference: https://docs.aws.amazon.com/AmazonCloudWatch/latest/events/EventTypes.html#events-for-services-not-listed).
All these calls seem to be available as input structs in the SDK but, for obvious reasons, don't depend on serde, meaning they cannot be plugged right in to type the Lambda CloudWatch events and be deserialized automatically (with camelCase renaming).
Are there any plans on supplying structs in a separate module (or perhaps even a separate crate) with serde (de-)serialization support? I think it's a bit out of scope for this project, but it also feels silly to maintain two almost identical structs in two different projects.
Platform/OS/Device
No response
Language Version
No response
The text was updated successfully, but these errors were encountered: