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

aws-sdk to devDepencies and region variable in the role #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

laardee
Copy link

@laardee laardee commented Oct 14, 2016

Very nice and simple example how to use DynamoDB with Serverless 👍

This PR includes couple small improvements

  • Moves aws-sdk to devDependecies - aws-sdk version 2.6.3 is included in the Lambda execution environment
  • Changes the IAM role to use ${self:provider.region} variable instead of hardcoded region

@paolini
Copy link

paolini commented Aug 31, 2017

I have changed the region in serverless.yml to eu-central-1 and when executing the server (no other modification) I get a AccessDeniedError when trying to access the database. Do you think your changes to the IAM role can fix this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants