Alexa-skills-kit-sdk-for-nodejs: Use Lambda callback parameter instead of context.fail context.succeed ?

Created on 2 Feb 2017  路  2Comments  路  Source: alexa/alexa-skills-kit-sdk-for-nodejs

I wondered, why I got no logs on CloudWatch for my alexa skill lambda function. Especially for the errors from my function.

As mentioned in the lambda documentation for nodejs handlers here: http://docs.aws.amazon.com/lambda/latest/dg/nodejs-prog-model-handler.html#nodejs-prog-model-handler-callback one should use the callback parameter to propagate the error to the CloudWatch logging.

At the moment the callback parameter is passed to the alexa handler function and saved into the _callback property, but it is never used. I think we should replace the _context.fail and _context.succeed function calls with the appropriate _callback calls.

Do you agree?
I will try to provide a PR for that, soon.

Best regards.

Most helpful comment

I've created a pull request: #59

All 2 comments

I've created a pull request: #59

Thank you for the PR, i'll track this via #59

closing #58

Was this page helpful?
0 / 5 - 0 ratings

Related issues

captnCC picture captnCC  路  6Comments

slesem picture slesem  路  4Comments

lucidokr picture lucidokr  路  5Comments

kevinwu picture kevinwu  路  3Comments

ancashoria picture ancashoria  路  6Comments