Skip to content
This repository has been archived by the owner on Oct 14, 2024. It is now read-only.

fix response header when lambda function returns errorMessage #102

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

Conversation

ezequielm
Copy link

@ezequielm ezequielm commented Sep 4, 2018

What did you implement:

Closes #101

How did you implement it:

When lambda function returns an errorMessage (that means the function handle the error) the invoke/local not reject the promise and registry/invoke set the correct header (X-Amz-Function-Error) for response of lambda invoke

How can we verify it:

  1. Go to examples folder, and start simulating lambda
  2. Open a new terminal and run aws lambda invoke --function-name docker-dev-lambdaError outfile.txt --endpoint-url http://localhost:4000

Todos:

  • Write tests
  • Write documentation
  • Fix linting errors
  • Provide verification config/commands/resources
  • Change ready for review message below

Is this ready for review?: YES

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

Successfully merging this pull request may close these issues.

1 participant