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

seems to fail when using alexa.registerHandlers and alexa.execute #293

Open
avishaan opened this issue May 23, 2017 · 2 comments
Open

seems to fail when using alexa.registerHandlers and alexa.execute #293

avishaan opened this issue May 23, 2017 · 2 comments
Labels

Comments

@avishaan
Copy link

When I do the simple alexa example that uses

  alexa.registerHandlers(handlers);
  alexa.execute();

I get the following type of failure when node-lambda run

image

code and test works fine in lambda itself

@abetomo
Copy link
Contributor

abetomo commented May 23, 2017

@codehatcher
(I could not build the environment of alexa, so write it by speculation. I am sorry if it did not become a reference.)

node-lambda ceased support of the writing not recommended by Lambda.
Reference: #223

However, alexa-sdk seems to be using context.succeed() which is not recommended by Lambda.
I think that it is an error that occurred because succeed() was not set in the context passed by var alexa = Alexa.handler(event, context);.

Should I make context.succeed() available for use?

@abetomo
Copy link
Contributor

abetomo commented Jun 28, 2017

@codehatcher I added a context method in #337

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

No branches or pull requests

3 participants