CODEWITHSUNDEEP
node.jstypescriptpromiseaxios
C.Hughes
C.Hughes

Reputation: 15

Retrieving JSON data from an Axios response

I'm using TypeScript to create a Node.js application and I want to retrieve JSON data from an external API. I have a demo version of the code I'm using, can't put my actual codebase up. 

private async getData() {
    return await Axios.get(
        `http://dummy.restapiexample.com/api/v1/employees`
    ).then(response => {
        return response.data;
    });
}

getReleaseResults() {
    this.getData().then(responseData => {
        responseData.data.data.forEach((element: any) => {
            console.log(element);
        });
    });
}

The error message I get is: (node:6068) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'forEach' of undefined at C:\Users\Caoilinn.Hughes\OneDrive\Documents\TypeScript Demos\Azure Test Result Email Extension\emailAzureExtension\app\out\js\apiCaller.js:43:36 at processTicksAndRejections (internal/process/task_queues.js:97:5) (node:6068) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:6068) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

If I don't have the for each and replace it with 

console.log(responseData.data.data)

I don't get any issues. FYI the result set has a data property hence the "data.data"

Upvotes: 1

Views: 5850

Answers (2)

satheesh m
satheesh m

Reputation: 451

Add catch block to get the error if there is any. What is the expected response ? try to log it.

return await Axios.get(
        `http://dummy.restapiexample.com/api/v1/employees`
    ).then(response => {
        console.log(response);
        return response.data;
    }).catch(err => console.log(err));

Upvotes: 0

Ashish Modi
Ashish Modi

Reputation: 7770

The problem is that you are already returning response.data from your get function and then again you are doing data.data which would not work.

see this. Remove extra data.

getReleaseResults() {
    this.getData().then(responseData => {
        responseData.data.forEach((element: any) => {
            console.log(element);
        });
    });
}

Upvotes: 2

Related Questions