在Express中正确使用app.set()?

时间:2018-03-02 16:21:59

标签: javascript node.js express middleware

在访问app.set()和app.get()之前,我无法访问我的快速中间件中的变量。但是,我想知道这是否适合使用app.set()以及是否有更好的方法来访问中间件内的变量。这是一个例子:

var express = require("express");
var http = require("http");
var https = require("https");
var s3Client = require("./lib/s3_wrapper");
var drillClient = require('./lib/drill_wrapper');
var fs = require("fs");

var app = express();
app.set("s3Client", s3Client);
app.set("drillClient", drillClient);

var adminParams = JSON.parse(fs.readFileSync("./conf/config.json"), "utf-8");
var adminBucket = adminParams.bucket;
var adminS3Client = new s3Client(adminParams);

app.use(function(req, res, next) {
    var accessKey = req.header("Authorization");
    adminS3Client.getObject({Bucket: adminBucket, Key: accessKey})
                 .then(function(results) {
                     req.config = JSON.parse(results.Body);
                     next();
                 })
                 .catch(function(err) {
                     console.log(err);
                     res.status(err.status).send(err.message);
                 });
});

app.get("/getObject/:bucket/:objectKey", function(req, res) {
    var params = {
        Bucket: req.params.bucket,
        Key: req.params.objectKey
    };
    var s3 = req.app.get("s3Client");
    var s3Client = new s3(req.config);
    s3Client.getObject(params)
            .then(function(results) {
                res.status(200).send(results.Body);
            })
            .catch(function(err) {
                res.status(500).send(err.message);
            });
});

正如您在app.get(“/ getObject”)路由中看到的那样,我正在使用req.app.get(“s3Client”)因为没有它我无法创建我的s3Client的新实例,因为s3Client在这一点上总是不确定的。谢谢你的时间。

1 个答案:

答案 0 :(得分:0)

我会冒险并说这不是一个好用的。你把整个模块放在那个变量里面?为什么?如果您需要在其他地方使用该模块,您只需再次使用它。如果您将所有内容都放在一个文件中,那么将其添加到顶部就足够了。

const s3Client = require("./lib/s3_wrapper");

app.get("/getObject/:bucket/:objectKey", function(req, res) {
  //s3Client was already defined.
  s3Client.something()
});

然后在某个其他文件中,某处。

//you import it again
const s3Client = require("./lib/s3_wrapper");

s3Client.something()