99x / serverless-dynamodb-client Goto Github PK

Hi,

Since the AWS SDK is available in the AWS Lambda execution environment, the aws-sdk package could be added to devDependencies instead, in order to reduce the package size.

What do you think?

Using this library doesn't seem to play nicely in the Lambda Node 18 runtime.

{ "errorType": "Runtime.ImportModuleError", "errorMessage": "Error: Cannot find module 'aws-sdk'\nRequire stack:\n- /var/task/node_modules/serverless-dynamodb-client/index.js\n- /var/task/handlers/profile.js\n- /var/runtime/index.mjs", "stack": [ "Runtime.ImportModuleError: Error: Cannot find module 'aws-sdk'", "Require stack:", "- /var/task/node_modules/serverless-dynamodb-client/index.js", "- /var/task/handlers/profile.js", "- /var/runtime/index.mjs", " at _loadUserApp (file:///var/runtime/index.mjs:1087:17)", " at async UserFunction.js.module.exports.load (file:///var/runtime/index.mjs:1119:21)", " at async start (file:///var/runtime/index.mjs:1282:23)", " at async file:///var/runtime/index.mjs:1288:1" ] }

This may need to be adjusted to import DynamoDB in a way that's compatible with aws-sdk v3
https://github.com/aws/aws-sdk-js-v3

It would be handy to allow using custom options when constructing a DynamoDB object.
For start it would be nice to support defining convertEmptyValues and port in serverless.yml.

Add CONTRIBUTING.md

Hey there!

I get a "TypeError: Plugin is not a constructor" when trying to run serverless offline start with serverless-dynamodb-client plugin enabled.

Version I used:

• serverless v1.32.0
• node v8.10
• serverless-offline v3.28.1
• serverless-dynamodb-local v0.2.35

On top of that I use typescript.

Thanks for the help

Add CODE_OF_CONDUCT.md

Scenario

Suppose you have configured plugins serverless-dynamodb-local, serverless-offline and serverless-dynamodb-client. Then, you execute commands

$ sls offline & [1]
$ sls invoke local -f function --path input.json [2]

Result

The AWS dynamodb instance will be used, not the local one.

Explanation

The reason is that:

• command [1] sets environment variable IS_OFFLINE not to the parent shell but to the sls offline process
• when command [2] is executed, it inherits the environment of the shell, which has no IS_OFFLINE defined. So, when require('serverless-dynamodb-client') is called, isOffline() always returns false.

Workaround

Execute

$ export IS_OFFLINE=true

before command [2]

The plugin doesn't detect local environments anymore because the IS_OFFLINE flag has been removed. See dherault/serverless-offline#768 (comment)

Please allow the port (8000) to be overridden via an environment variable, to make running multiple related services on the same machine easier.

Thanks!