ждите обещание перед обратным вызовом в async.each

Question

ждите обещание перед обратным вызовом в async.each

1

router.post('/runCommand', async function(req, res){
  let results = [];
  async.each(req.body.requests, async function(request, callback){
    const data = await connect(request.command)
    await results.push(data);
    await callback(null);
  }, function(err){
    if (!err) {
      res.send(202, results)
    }
  })
})

Res.send никогда не происходит, и обратный вызов, кажется, происходит до завершения подключения. Connect успешно возвращает обещание, потому что это

router.get('/topics', async function(req, res) {
  console.log('in get');
  const data = await connect(req.body.command);
  await res.send(data);
});

работает отлично. Но включение async.each для запуска нескольких команд кажется сломанным. Я знаю, что это проблема с тем, как я вызываю функцию обратного вызова async.each, но исследования не помогли мне, как я должен ее называть. Возможно ли использовать .then() после ожидания обещания?

function connect(command){
  return new Promise(function(resolve) {
  let host = {
        server: {
          host: "host",
          port: "port",
          userName: "user",
          password: config.Devpassword
        },
        commands: [ command ]
      };
  var SSH2Shell = require ('ssh2shell'),
  //Create a new instance passing in the host object
  SSH = new SSH2Shell(host),
  //Use a callback function to process the full session text
  callback = function(sessionText){
    console.log(sessionText)
    resolve(sessionText);
  }
  SSH.connect(callback);
  })
};

Noah Rolf 26 окт. 2018, в 18:23

Источник

0

Ваш обратный вызов async.each() когда-либо вызывался? Если да, то что было ошибкой? Здесь недостаточно информации, чтобы определить, где вы идете не так. Вы уверены, что req.body вашего POST-запроса будет отформатировано как { requests: [ { command: ... }, ...] } ?
Patrick Roberts 26 окт. 2018, в 16:31
0

Я могу сказать по логам консоли в функции подключения, что это происходит для каждой команды в req. и да, это правильный формат. Проблема, с которой я сталкиваюсь, заключается в том, когда выполняется обратный вызов. Я не жду соединения, чтобы закончить бежать. поэтому async.each проходит через каждую команду в req до того, как массив результатов будет правильно определен. Хотя я не уверен, почему res.send никогда не происходит. Я предполагал, что это просто отправит и массив невыполненных обещаний, но на самом деле никогда не отправляет вообще
Noah Rolf 26 окт. 2018, в 16:37
0

Пожалуйста, предоставьте источник для connect() , отредактировав свой ответ. Кажется, проблема в этом.
Patrick Roberts 26 окт. 2018, в 16:40
0

И вы получаете console.log(sessionText) для каждого request в req.body.requests ?
Patrick Roberts 26 окт. 2018, в 16:45
0

@PatrickRoberts Я добавил источник для connect() хотя я не верю, что проблема есть. Возвращает обещание в порядке. Когда я запускаю только одну команду с другой функцией, она должным образом ожидает ее завершения перед повторной отправкой текста сеанса. Я думаю, что проблема заключается в том, как я вызываю callback () из async.each
Noah Rolf 26 окт. 2018, в 16:45
0

да, я попытался передать 3 команды вниз, и журнал консоли произошел для всех 3
Noah Rolf 26 окт. 2018, в 16:45
0

Попробуйте callback() вместо callback(null) . Я был бы удивлен, если это работает, но это единственное, что кажется возможным изменить ситуацию здесь. Вы неправильно используете await в нескольких местах, но это не влияет на то, что должно произойти.
Patrick Roberts 26 окт. 2018, в 16:50
0

@PatrickRoberts, как ни странно, удаление null, по крайней мере, сделало так, чтобы вызывался res.send. Но он отправил только пустой массив вместо массива из сессионных текстов. Так что callback() определенно выполняется до завершения подключения. Можно ли переформатировать подключение к const data = await connect().then( callback() )
Noah Rolf 26 окт. 2018, в 17:09
0

Вы уверены, что источник, который вы опубликовали для router.post() это то, что вы на самом деле используете? Я не хочу звучать так, будто задаю очевидный вопрос; Я спрашиваю, потому что для просителей характерно удалять код в размещенных фрагментах по соображениям краткости или по личным причинам, и иногда эти удаления приводят к тому, что минимальный пример ведет себя по-другому.
Patrick Roberts 26 окт. 2018, в 17:15
0

И нет, предлагаемое вами изменение не будет работать по нескольким причинам.
Patrick Roberts 26 окт. 2018, в 17:16
0

Да, я дважды проверил. Он идентичен даже по именам функций и переменных
Noah Rolf 26 окт. 2018, в 17:16
0

Я все еще пытаюсь выяснить эту проблему. Это кажется мне очень странным.
Patrick Roberts 26 окт. 2018, в 17:28

Показать ещё 10 комментариев

Теги:

javascript

callback

async-await

es6-promise

async.js

1 ответ

Ещё вопросы

Ваш обратный вызов async.each() когда-либо вызывался? Если да, то что было ошибкой? Здесь недостаточно информации, чтобы определить, где вы идете не так. Вы уверены, что req.body вашего POST-запроса будет отформатировано как { requests: [ { command: ... }, ...] } ?
Я могу сказать по логам консоли в функции подключения, что это происходит для каждой команды в req. и да, это правильный формат. Проблема, с которой я сталкиваюсь, заключается в том, когда выполняется обратный вызов. Я не жду соединения, чтобы закончить бежать. поэтому async.each проходит через каждую команду в req до того, как массив результатов будет правильно определен. Хотя я не уверен, почему res.send никогда не происходит. Я предполагал, что это просто отправит и массив невыполненных обещаний, но на самом деле никогда не отправляет вообще
Пожалуйста, предоставьте источник для connect() , отредактировав свой ответ. Кажется, проблема в этом.
И вы получаете console.log(sessionText) для каждого request в req.body.requests ?
@PatrickRoberts Я добавил источник для connect() хотя я не верю, что проблема есть. Возвращает обещание в порядке. Когда я запускаю только одну команду с другой функцией, она должным образом ожидает ее завершения перед повторной отправкой текста сеанса. Я думаю, что проблема заключается в том, как я вызываю callback () из async.each
да, я попытался передать 3 команды вниз, и журнал консоли произошел для всех 3
Попробуйте callback() вместо callback(null) . Я был бы удивлен, если это работает, но это единственное, что кажется возможным изменить ситуацию здесь. Вы неправильно используете await в нескольких местах, но это не влияет на то, что должно произойти.
@PatrickRoberts, как ни странно, удаление null, по крайней мере, сделало так, чтобы вызывался res.send. Но он отправил только пустой массив вместо массива из сессионных текстов. Так что callback() определенно выполняется до завершения подключения. Можно ли переформатировать подключение к const data = await connect().then( callback() )
Вы уверены, что источник, который вы опубликовали для router.post() это то, что вы на самом деле используете? Я не хочу звучать так, будто задаю очевидный вопрос; Я спрашиваю, потому что для просителей характерно удалять код в размещенных фрагментах по соображениям краткости или по личным причинам, и иногда эти удаления приводят к тому, что минимальный пример ведет себя по-другому.
И нет, предлагаемое вами изменение не будет работать по нескольким причинам.
Да, я дважды проверил. Он идентичен даже по именам функций и переменных
Я все еще пытаюсь выяснить эту проблему. Это кажется мне очень странным.

Patrick Roberts · Accepted Answer · 2018-10-26T16-30-00.000Z

Хотя вы можете продолжать async.each() больше времени работе async.each(), я рекомендую просто отбросить его и перейти исключительно с синтаксисом async/await который значительно упрощает ваш код:

router.post('/runCommand', async function (req, res) {
  try {
    const results = await Promise.all(
      req.body.requests.map(({ command }) => connect(command))
    );

    res.send(202, results);
  } catch ({ message, stack }) {
    res.send(500, { error: message, stack });
  }
})

Если посмотреть на документацию ssh2shell, я думаю, что ваша функция connect может быть улучшена для лучшей читаемости и обработки ошибок:

const SSH2Shell = require('ssh2shell');

function connect (command) {
  return new Promise((resolve, reject) => {
    const host = {
      server: {
        host: 'host',
        port: 'port',
        userName: 'user',
        password: config.Devpassword
      },
      commands: [command]
    };
    //Create a new instance passing in the host object
    const SSH = new SSH2Shell(host);

    SSH.on('error', reject);
    SSH.connect(resolve);
  });
}

Пожалуйста, не стесняйтесь комментировать, если это все еще не работает для вас.

Спасибо! Вы были на 100% правы об упрощении. Я определенно буду использовать это, но могу посмотреть на async.each, когда у меня будет больше времени.