Node.js ReadLine не ждет полной строки в соединениях сокета?

Когда я подключаюсь telnet к порту 7001 и начинаю вводить текст, он сразу же возвращается к мне, прежде чем я когда-либо нажимаю на ввод.

Я также попробовал .question() и получил те же результаты... Обратный вызов запускается при получении любых данных, не дожидаясь окончания -строчный символ.

Изменить: Это становится еще более странным. Когда я тестирую с помощью клиента telnet Windows, я получаю поведение, о котором я говорил выше. Однако, если я тестирую PuTTY как клиент, ReadLine работает даже в Windows. Я сделал некоторые захваты пакетов. Может быть, кто-то может пролить свет на это? Непредвиденные строки - это данные от клиента. Отступы - это ответы сервера.

Ответы

Ответ 1

Это ошибка в node.js, интерфейс ReadLine вызывает _normalWrite() на каждом событии данных, а _normaWrite имеет комментарий, что он должен попытаться сломать строки новой строки, но в настоящее время он просто вызывает _onLine().

Что-то вроде этого должно исправить это для вас:

i._normalWrite = function(b) {
    if(b == undefined) {
        return;
    }
    if(!this._line_buffer) {
        this._line_buffer = '';
    }
    this._line_buffer += b.toString();
    if(this._line_buffer.indexOf('\n') !=-1 ) {
        var lines = this._line_buffer.split('\n');
        // either '' or the unfinished portion of the next line
        this._line_buffer = lines.pop();
        lines.forEach(function(line) {
            this._onLine(line + '\n');
        }, this);
    }
};

Я не тестировал это, возможно, потребуется также принять \r. Пожалуйста, дайте мне знать, если это сработает для вас, если это так, тогда один из нас должен отправить node запрос на перенос с ним.

Ответ 2

Альтернативное решение моей проблемы... Мне просто нужно было получить какое-то событие линии каждый раз, когда появилась новая строка из потока. Поскольку мне не нужно все, что поставляется с readline, я нашел этот фрагмент с помощью TooTallNate здесь: https://gist.github.com/1785026

/**
 * By TooTallNate, originally posted at https://gist.github.com/1785026
 * A quick little thingy that takes a Stream instance and makes
 * it emit 'line' events when a newline is encountered.
 *
 *   Usage:
 *   ‾‾‾‾‾
 *  emitLines(process.stdin)
 *  process.stdin.resume()
 *  process.stdin.setEncoding('utf8')
 *  process.stdin.on('line', function (line) {
 *    console.log(line event:', line)
 *  })
 *
 */

function emitLines (stream) {
  var backlog = ''
  stream.on('data', function (data) {
    backlog += data
    var n = backlog.indexOf('\n')
    // got a \n? emit one or more 'line' events
    while (~n) {
      stream.emit('line', backlog.substring(0, n))
      backlog = backlog.substring(n + 1)
      n = backlog.indexOf('\n')
    }
  })
  stream.on('end', function () {
    if (backlog) {
      stream.emit('line', backlog)
    }
  })
}

Это было опубликовано как комментарий к запросу Nathan pull здесь: https://github.com/joyent/node/pull/3059