Alexa nodejs accessing url from Amazon Lambda

Question

I've created a simple skill for Alexa based on this example: https://github.com/alexa/skill-sample-nodejs-fact/blob/en-US/lambda/custom/index.js

Now, I'd like the script to log something on a different server when GetNewFactIntent is called.

This is what I'm trying to do but have an issue with this which is not what it should be in the http.get callback.

'GetNewFactIntent': function () {
//var thisisit = this;
http.get("http://example.com", function(res) {
  //console.log("Got response: " + res.statusCode);
  const factArr = data;
  const factIndex = Math.floor(Math.random() * factArr.length);
  const randomFact = factArr[factIndex];
  const speechOutput = GET_FACT_MESSAGE + randomFact;

  this.response.cardRenderer(SKILL_NAME, randomFact);
  this.response.speak(speechOutput);
  this.emit(':responseReady');
}).on('error', function(e) {
  //console.log("Got error: " + e.message);
});
},

What this needs to be replaced with in the example above for this to work?

I can't see an immediate problem with that code. Can you add additional information about exactly how it is failing? More context is needed. — i336_

NikxDa NikxDa · Accepted Answer · 2017-11-11T11:02:13

this will not be what you think it is, because you are in the context of the callback function. There are two possible solutions:

Use an arrow function instead. An arrow function preserves the this variable of the scope it is being used in: function () { ... } -> () => { }.
Declare var self = this; outside of the callback and then replace your this inside the callback with your self variable.

Example:

function getStuff () {
    var self = this;
    http.get (..., function () {
        // Instead of this, use self here
    })
}

For further information, see: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/this

Alexa nodejs accessing url from Amazon Lambda

1 Answers