Express 中间件调用多次

Question

我查看了与此主题相关的其他帖子，但在我的代码中找不到问题。

const myMiddleware = (fn) => {
    return (req, res, next) => {
        var fullUrl = req.protocol + '://' + req.get('host') + req.url;
        console.log(fullUrl)
        next()
    }
}

const app = express()

app.use('/dist', express.static(__dirname + '/client/dist'))
app.use('/static', express.static(__dirname + '/client/static'))

app.use(bodyParser.urlencoded({ extended: false }))
app.use(cookieParserMiddleware())
app.use(passport.initialize())

const server = https.createServer(options, app)

app.get('/', myMiddleware((req, res) => {
    res.sendFile(__dirname + '/client/dist/index.html')
}))

app.all('*', (req, res) => {
    res.redirect('/')
})

server.listen(8080, function listening() {
    console.log('Listening on %d', server.address().port)
})

没有myMiddleware 上的'/' 路径，一切都按预期工作。将myMiddleware 附加为app.get('/', myMiddleware((req, res) => {，然后多次调用myMiddleware 而没有调用res.sendFile(__dirname + '/client/dist/index.html')。

---

编辑：下面的错误已通过 jfriend00 的解决方案修复。被多次调用的中间件仍然存在。原因是图标和其他一些资产没有被app.use('/static', express.static(__dirname + '/client/static')) 行捕获。修复路径也解决了第一个错误

除此之外，我尝试删除下面的部分，但该应用程序根本无法运行。我的猜测是这里有 2 个错误。

app.all('*', (req, res) => {
    res.redirect('/')
})

我发布了一张图片，说明删除 app.all('*'..) 后的外观

Answer 1

我在这里做个猜测。

改变这个：

app.get('/', myMiddleware((req, res) => {
    res.sendFile(__dirname + '/client/dist/index.html')
}));

到这里：

app.get('/', myMiddleware(), (req, res) => {
    res.sendFile(__dirname + '/client/dist/index.html')
}));

这样您的res.sendFile() 在myMiddleware 调用next() 之后实际上会被调用。

---

然后，删除 fn 参数，这样你就有了（不会改变执行，但会删除误导性和未使用的参数）：

const myMiddleware = () => {
    return (req, res, next) => {
        var fullUrl = req.protocol + '://' + req.get('host') + req.url;
        console.log(fullUrl)
        next()
    }
}