节点HTTP请求永远挂起

时间:2017-09-18 14:06:19

标签: javascript node.js http request

我们有一个Node.js脚本,每分钟运行一次,以检查我们的应用程序的状态。通常,它工作得很好。如果服务已启动,则会以0退出。如果服务已关闭,则退出为1.一切顺利。

但每隔一段时间,它就会停止。控制台报告“调用状态API ...”并无限期地停止。在Node的内置两分钟超时时,它甚至没有超时。没有错误,没有。它只是坐在那里,等待,永远。这是一个问题,因为它会阻止以下状态检查作业运行。

此时,我的整个团队已经看过它,我们都不知道什么环境可以让它挂起。我们已经建立了一个从头到尾的超时,以便我们可以继续下一个工作,但这实际上会跳过状态检查并产生盲点。所以,我向你们提出这个问题。

这是脚本(删除了姓名/网址):

#!/usr/bin/env node

// SETTINGS: -------------------------------------------------------------------------------------------------
/** URL to contact for status information. */
const STATUS_API = process.env.STATUS_API;

/** Number of attempts to make before reporting as a failure. */
const ATTEMPT_LIMIT = 3;

/** Amount of time to wait before starting another attempt, in milliseconds. */
const ATTEMPT_DELAY = 5000;

// RUNTIME: --------------------------------------------------------------------------------------------------
const URL = require('url');
const https = require('https');

// Make the first attempt.
make_attempt(1, STATUS_API);

// FUNCTIONS: ------------------------------------------------------------------------------------------------
function make_attempt(attempt_number, url) {
    console.log('\n\nCONNECTION ATTEMPT:', attempt_number);
    check_status(url, function (success) {
        console.log('\nAttempt', success ? 'PASSED' : 'FAILED');

        // If this attempt succeeded, report success.
        if (success) {
                console.log('\nSTATUS CHECK PASSED after', attempt_number, 'attempt(s).');
                process.exit(0);
        }

        // Otherwise, if we have additional attempts, try again.
        else if (attempt_number < ATTEMPT_LIMIT) {
            setTimeout(make_attempt.bind(null, attempt_number + 1, url), ATTEMPT_DELAY);
        }

        // Otherwise, we're out of attempts. Report failure.
        else {
            console.log("\nSTATUS CHECK FAILED");
            process.exit(1);
        }
    })
}

function check_status(url, callback) {
    var handle_error = function (error) {
        console.log("\tFailed.\n");
        console.log('\t' + error.toString().replace(/\n\r?/g, '\n\t'));
        callback(false);
    };

    console.log("\tCalling status API...");
    try {
        var options = URL.parse(url);
        options.timeout = 20000;
        https.get(options, function (response) {
            var body = '';
            response.setEncoding('utf8');
            response.on('data', function (data) {body += data;});
            response.on('end', function () {
                console.log("\tConnected.\n");
                try {
                    var parsed = JSON.parse(body);
                    if ((!parsed.started || !parsed.uptime)) {
                        console.log('\tReceived unexpected JSON response:');
                        console.log('\t\t' + JSON.stringify(parsed, null, 1).replace(/\n\r?/g, '\n\t\t'));
                        callback(false);
                    }
                    else {
                        console.log('\tReceived status details from API:');
                        console.log('\t\tServer started:', parsed.started);
                        console.log('\t\tServer uptime:', parsed.uptime);
                        callback(true);
                    }
                }
                catch (error) {
                    console.log('\tReceived unexpected non-JSON response:');
                    console.log('\t\t' + body.trim().replace(/\n\r?/g, '\n\t\t'));
                    callback(false);
                }
            });
        }).on('error', handle_error);
    }
    catch (error) {
        handle_error(error);
    }
}

如果你们中的任何人可以看到任何可能在没有输出或超时的情况下挂起的地方,那将非常有帮助!

谢谢你, 詹姆斯坦纳

编辑: p.s.我们直接使用https而不是request,这样我们就不需要在脚本运行时进行任何安装。这是因为脚本可以在没有自定义安装的情况下在分配给Jenkins的任何构建计算机上运行。

2 个答案:

答案 0 :(得分:2)

在你的回复中回调你没有检查状态..

.on('error', handle_error);用于连接服务器时发生的错误,状态代码错误是服务器在成功连接后响应的错误。

通常情况下,200状态响应是您对成功请求的期望。

所以你的http.get处理这个的一个小mod应该做..

例如

https.get(options, function (response) {
  if (response.statusCode != 200) {
    console.log('\tHTTP statusCode not 200:');
    callback(false);
    return; //no point going any further
  }
  ....

答案 1 :(得分:0)

您不错过.end()吗?

http.request(options, callback).end()

类似here的解释。