Как сделать множественные request на node.js?

Question

[Aleksandr]

Есть массив с ID пользователей. Необходимо через API сайта получить данные этих пользователей. Я делаю это таким образом:

const request = require(`request-promise`);
let ids = [2113811902, 2113811976, 2112091110 /* , ... */];

Promise
    .all(ids.map(id => {
        return request
            .get(`http://site.com/api/user/${id}`)
            .then(user => {
                return user;
            })
            // Что бы Promise.all обработал все запросы
            // обрабатываем ошибку каждого запросы
            // и возвращаем результат с ошибкой
            // и с ID, с которым была ошибка
            .catch(error => {
                return {
                    error: error.message,
                    userId: id
                };
            });
    }))
    .then(result => {
        console.log(result);
    })
    .catch(error => {
        console.error(error.message);
    });

Правильный ли я подход использую? Как лучше сделать?

Comments

[Анатолий Жаров]

Правильный ли я подход использую? Правильный
вот это только лишнее

.then(user => {
    return user;
})

[Aleksandr]

Анатолий Жаров, Лишнее потому что Promise.all и так получит результаты если все ок? В этом участке кода можно обработку полученного результата сделать, например.

[Алексей Ступеньков]

При определенном количестве вы можете устроить DDOS сайта.

[Endy]

Алексей Ступеньков, это из-за .map?

[Aleksandr]

Endy, это из-за того, что одновременно посылаю запросы.

[Endy]

Aleksandr, а как правильно? можешь скинуть что-то по теме почитать?

[Aleksandr]

Endy, все зависит от API сервера. Есть ограничения на кол-во запросов в секунду, есть ограничение на кол-во ошибок. Правильно, узнать все нюансы и следовать правилам =)