Как прервать js запрос к серверу?

Question

[danilr]

Происходит запрос через стороннюю библиотеку к API, но какие то запросы просто зависают и node js часами может ждать ответ запроса, но он так и не приходит и ошибок никаких не возникает.
Как сбросить этот запрос и продолжить выполнение js скрипта если в течении 3 минут не получен ответ? Запросы делаются в цикле.

for (let i=0; i < 300; i++) {
     
      try {
        const timerId = setTimeout(() => {
          console.log('in setTimeout continue');
          continue;
        }, 1000 * 60 * 3);
        await Order(i); // то самое стороннее апи
      }
      catch(e) {
        console.log('error_text:', e)
      }
      finally{
        clearTimeout(timerId);
      }
    }

так пробовал через setTimout делать continue для продолжения цикла. Правильно ли и сработает ли? Или есть более лучший вариант?

Answer 1

[danilr]

Promise.any не поддерживается в nodejs 12, поэтому решение через Promise.race:

let timerId = 0; // вынес так как не знал как таймаут потом в каждой итерации сбрасывать

      try {

        const firstPromiseResult = await Promise.race([
          new Promise((resolve) => {
            timerId = setTimeout(()=> {
              resolve('TIMEOUT')
            }, 1000 * 60 * 4) // время через которое мы идём дальше по циклу( тут его не видно(цикла))
          }),
          order(Id) // наш запрос к стороннему апи, тут без await т.к. зависает и проблема остаётся
        ]);

      }
      catch(e) {
        console.log('error_text:', e)
      }
      finally{
        clearTimeout(timerId); // сбрасываем таймаут в любом случае, чтобы не перегружать систему
      }
    }

Answer 2

[Сергей Соколов]

По-хорошему суметь настроить веб-запросер в той библиотеке, указав ему таймаут веб-соединения.

Можно через Promise.any() ждать, что выполнится раньше: запрос или таймаут.

Что-то типа

const TIMEOUT = "TIMEOUT";

for (let i=0; i < 300; i++) {
  await Promise.any([
    Order(i), // то самое стороннее апи
    new Promise((resolve) => setTimeout(() => resolve(TIMEOUT), 3 * 6e4))
  ])
  .then((result) => {
    if (result === TIMEOUT) {
      console.log(TIMEOUT);
    } else {
      console.log("Order fulfilled");
    }
  })
}

Тут ещё видимо надо ловить catch() в цепочке Order(), и далее ресолвить со значением ошибки, т.к. reject'нутные промисы не влияют на исход Promise.any(), если только вообще все участники откажутся

Comments

[danilr]

Сергей Соколов Решение по идее должно работать, только непонятно где тут сбрасывать и очищать setTimeout с помощью clearTimeout(id) ? У меня будет десятки тысяч запросов, надо сбрасывать будет их в каждой итерации.
И да, then мне тут не подходит, нужно всё через async await

[Сергей Соколов]

danilr, сбрасывать таймаут не понадобится — он сработает позже, но это уже всем безразлично.

then() дружит с async / await.

await это «синтаксический подсластитель», ожидающий разрешения промиса. Кто запрещает писать

const f = async (n) => {
  await new Promise((res) => setTimeout(res, n))
    .then(() => console.log("OK"));
  console.log("Have you seen OK above?");
}

f(1000);

[danilr]

Сергей Соколов таймаут то сработает, но это очень сильно будет нагружать систему - представьте 10000 таймаутов делают отсчёт