Skip to content
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

[Feature]: Support the AWS.DynamoDB.Converter methods #751

Open
1 task
obones opened this issue Jan 3, 2024 · 0 comments
Open
1 task

[Feature]: Support the AWS.DynamoDB.Converter methods #751

obones opened this issue Jan 3, 2024 · 0 comments
Labels
enhancement New feature or request p2 This is a standard priority issue

Comments

@obones
Copy link

obones commented Jan 3, 2024

Self-service

  • I'd be willing to implement this feature

Problem

As I'm processing DynamoDB notifications with a lambda function, I have the following code:

const keys = AWS.DynamoDB.Converter.unmarshall(record.dynamodb.Keys);

But this is not being converted, just left as is.
Note that the const AWS = require("aws-sdk"); line is not removed in that case leading to a non running code.

Solution

Ideally, this should add the following import

const { unmarshall } = require("@aws-sdk/util-dynamodb");

and change the code to this

const keys = unmarshall(record.dynamodb.Keys);

Alternatives

Converting manually was easy enough in my case

Additional context

No response

@obones obones added the enhancement New feature or request label Jan 3, 2024
@trivikr trivikr added the p3 This is a minor priority issue label Jan 12, 2024
@trivikr trivikr added p2 This is a standard priority issue and removed p3 This is a minor priority issue labels Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request p2 This is a standard priority issue
Projects
None yet
Development

No branches or pull requests

2 participants