Node.js服务器停止响应

Question

我正在处理node.js HTTPS请求。 我已经进行了足够的研究，还检查了所有内容，但我仍然认为我缺少某些内容。 我要实现的是，当有人向我的URL发送HTTPS GET请求时，我必须向他们发送一个文件作为响应，以便他们可以下载。 一切正常，我已经实现了“ FS”，“ HTTPS”，并且也以适当的方式编写，但是当我平均每40个请求中的每1000个请求失败后，在我的URL上发送大量流量时。

我不确定为什么会这样，我也设置了这个：

var https = require('https');
https.globalAgent.maxSockets = Infinity;

谁能帮我理解和解决问题？ 提前致谢！

安韦什

编辑 ：根据要求，我正在添加我的代码，由于隐私原因，我删除了几行代码。

var https = require('https');
https.globalAgent.maxSockets = Infinity;
//var port = process.env.port || 1337;
fileSystem = require('fs'),
fileToWriteLog = require('fs'),
    path = require('path');
var mime = require('mime');
var cors = require('cors');


var XMLHttpRequest = require("xmlhttprequest").XMLHttpRequest;

var express = require('express'),
    app = module.exports.app = express();

app.use(cors());


// Created to use HTTPS
var options = {
    pfx: fileSystem.readFileSync('Certificate/key.pfx'),
    passphrase: 'XXX'   
};

var server = https.createServer(options,app);
var io = require('socket.io').listen(server);  //pass a https.Server instance
var port = 8080;
console.log(process.env.PORT);
server.listen(port,"0.0.0.0", function () {
    console.log("Secure Express server listening on port " + port);
});  //listen on port 8082

// routing

app.get('/stream/:patam1/:param2', function (request, response) {    
        var patam1 = request.param("patam1");
        var param2 = decodeURIComponent(request.param("param2"));


                var filePath = "XXXX" + param2;
                console.log(filePath);

                var stat = fileSystem.statSync(filePath);
                response.writeHead(200, {
                    'Content-Type': 'XXX', 
                    'Content-Length': stat.size
                });

                var readStream = fileSystem.createReadStream(filePath);
                readStream.on('data', function (data) {                                 
                    var flushed = response.write(data);
                    // Pause the read stream when the write stream gets saturated
                    if (!flushed)
                        readStream.pause();
                });
                response.on('drain', function () {                                  
                    // Resume the read stream when the write stream gets hungry 
                    readStream.resume();
                });
                // readStream.on('end', function () {                        
                     // response.end();
                // });          

});

Answer 1

一般问题

我不知道您的服务器在什么操作系统上运行。 如果是Linux或Windows，请查看有关如何增加并发连接数量的答案：

• http服务器可以处理多少个客户端？
• Node.js Express Windows Max连接设置

您的问题可能出在您的客户端（在注释中写的那样在Windows上运行）或服务器（您未指定其OS）上。 同样，问题可能出在您的程序或操作系统上，甚至是客户端和服务器之间的其他一些问题，例如代理等。

您可以使用Apache ab命令来测试您的服务器，以便至少可以消除发送请求的程序在此处出错的可能性。 例如：

ab -n 10000 -c 1000 -k http://localhost:8080/

这将总共发送10000个请求，一次发送1000个。

请参见本关于如何测试与服务器的详细信息ab 。

封锁作业

这是您的问题：

var stat = fileSystem.statSync(filePath);

当您在处理程序中使用阻塞（同步）调用时，您将始终遇到并发连接问题。 我不知道这是否是唯一的问题，但是仅当您希望服务器很好地处理并发连接时，才需要使用异步调用。 使用stat代替statSync 。

同步功能非常适合程序启动时的一次性工作，就像您使用readFileSync来填充options ，而不是在事件处理程序内部运行。

请参阅此答案，以获取有关如何以多种不同方式（使用Express，不使用Express等）从磁盘上提供文件而无任何同步调用的信息。 看到：

• 如何使用Node.js提供图片

流

您根本不需要使用stat （或statSync ）。 您不必设置Content-length标头，因为Node可以使用分块编码，而您只需流数据即可。

我会改变这一点：

app.get('/stream/:patam1/:param2', function (request, response) {    
        var patam1 = request.param("patam1");
        var param2 = decodeURIComponent(request.param("param2"));

                var filePath = "XXXX" + param2;
                console.log(filePath);

                var stat = fileSystem.statSync(filePath);
                response.writeHead(200, {
                    'Content-Type': 'XXX', 
                    'Content-Length': stat.size
                });

                var readStream = fileSystem.createReadStream(filePath);
                readStream.on('data', function (data) {                                 
                    var flushed = response.write(data);
                    // Pause the read stream when the write stream gets saturated
                    if (!flushed)
                        readStream.pause();
                });
                response.on('drain', function () {                                  
                    // Resume the read stream when the write stream gets hungry 
                    readStream.resume();
                });
                // readStream.on('end', function () {                        
                     // response.end();
                // });          

});

对于这样的事情：

app.get('/stream/:patam1/:param2', function (request, response) {    
        var patam1 = request.param("patam1");
        var param2 = decodeURIComponent(request.param("param2"));

                var filePath = "XXXX" + param2;
                console.log(filePath);

                response.set('Content-Type', 'XXX');

                var readStream = fileSystem.createReadStream(filePath);
                readStream.on('open', function () {
                    response.set('Content-Type', 'XXX');
                    readStream.pipe(response);
                });
                readStream.on('error', function () {
                    response.set('Content-Type', 'text/plain');
                    response.status(404).end('Not found');
                });    
});

请参阅我在GitHub上的node-static-http-servers项目，尤其是express示例 。 它或多或少地完成了您在此所做的工作。 在此进行更详细的说明。

路径遍历

我会提出的另一个建议是保护您的服务器免受路径遍历攻击。 因此，当您使用以下命令设置文件名时：

var filePath = "XXXX" + param2;

最好使用path模块：

var filePath = path.join("XXXX", param2);

然后测试它是否不在您要提供的目录之外，例如：

if (filePath.indexOf('XXXX/') !== 0) {
    return response.status(403).end('Forbidden');
}

indexOf的/斜杠很重要。

异步统计信息示例

代替这个：

         var filePath = "XXXX" + param2;
            console.log(filePath);

            var stat = fileSystem.statSync(filePath);
            response.writeHead(200, {
                'Content-Type': 'XXX', 
                'Content-Length': stat.size
            });

            var readStream = fileSystem.createReadStream(filePath);
            readStream.on('data', function (data) {                                 
                var flushed = response.write(data);
                // Pause the read stream when the write stream gets saturated
                if (!flushed)
                    readStream.pause();
            });
            response.on('drain', function () {                                  
                // Resume the read stream when the write stream gets hungry 
                readStream.resume();
            });

您可以这样写：

        // check for path traversal as I described above:
        var filePath = path.join("XXXX", param2);
        if (filePath.indexOf('XXXX/') !== 0) {
            return response.status(403).end('Forbidden');
        }
        console.log(filePath);

        fileSystem.stat(filePath, function (err, stat) {

            if (err) {
                response.set('Content-Type', 'text/plain');
                return response.status(404).end('Not found');
            }

            var readStream = fileSystem.createReadStream(filePath);

            readStream.on('open', function () {
                response.set('Content-Type', 'XXX');
                response.set(Content-Length': stat.size);
                readStream.pipe(response);
            });
            readStream.on('error', function () {
                response.set('Content-Type', 'text/plain');
                response.status(404).end('Not found');
            });

        });