express.static() 的 Express 中间件没有正确路由并且抛出 next() 不是一个函数

Posted 2023-02-25

【中文标题】express.static() 的 Express 中间件没有正确路由并且抛出 next() 不是一个函数

【英文标题】：Express middleware for express.static() not routing correctly and throwing next() is not a function

【发布时间】：2021-01-15 01:55:12

【问题描述】：

尽管在我编写和测试它时它在我的本地机器上正常运行，但在使用 Docker 部署到我们的 UAT 环境时似乎失败了。我似乎无法追查为什么它给了我以下信息：

next() is not a function

我似乎也遇到了路由问题（再次仅在部署时），即使当路由是 /media/public 时，它似乎仍然会命中 /media 路由。

依赖

"dotenv": "^8.2.0",
"express": "^4.17.1",
"jsonwebtoken": "^8.5.1"

server.js

require('dotenv').config();
const express = require('express');
const  join  = require('path');
const  fileExists, isAuthenticated  = require('./helpers/utils');
const  PORT, ROOT_SHARE  = process.env;
const app = express();

global.__basedir = __dirname

app.use('/robots.txt', function (_req, res) 
    res.type('text/plain');
    res.send('User-agent: *\nDisallow:/');
);

app.use('/media/public', [fileExists, express.static(join(__dirname, ROOT_SHARE, '/public'))]);
app.use('/media', [isAuthenticated, express.static(join(__dirname, ROOT_SHARE))]);

app.listen(PORT, () => console.info(`[File Server] : running on $PORT`));

utils.js

const  existsSync  = require('fs');
const  verify  = require('jsonwebtoken');
const  join  = require('path');
const  ACCESS_SECRET, NODE_ENV   = process.env;

const fileExists = async (req, res, next) => 
    let mediaFile = (NODE_ENV === 'local') ? join(__basedir, req.baseUrl, req.path).replace('\\src','') : req.originalUrl;
    console.log(mediaFile);
    if (!existsSync(mediaFile)) 
        console.log('NOT FOUND')
        return res.status(404).send('NOT FOUND');
    

    return next();


const isAuthenticated = async (req, res, next) => 
    const accessToken = valueOrNull((req.cookies && req.cookies['x-access-token']) || (req.headers && req.headers['x-access-token']));
    
    if (!accessToken) 
        console.log('accessToken not found')
        return res.status(401).send('UNAUTHORIZED');
    

    try 
        fileExists(req);
        const  user  = verify(accessToken, ACCESS_SECRET);

        if (user) 
            console.log('VALID USER');
        
     catch (err) 
        console.log(err)
        return res.status(401).send('UNAUTHORIZED');
    

    return next();
;

const valueOrNull = (value) => 
    return (typeof value == 'undefined' || value === null || value === '') ? undefined : value;


module.exports = 
    fileExists,
    isAuthenticated

.env

ROOT_SHARE=../media

Dockerfile

FROM node:latest

RUN mkdir -p /media

WORKDIR /app

COPY package.json .
COPY package-lock.json .

RUN npm install && npm cache clean --force --loglevel=error

RUN npm install -g pm2

COPY src /app/src

EXPOSE 3000

ENTRYPOINT ["node", "src/server.js"]

准确的错误

任何帮助将不胜感激

【参考方案1】：

您的fileExists 被用作中间件和isAuthenticated 中调用的函数——在isAuthenticated 中它没有被传递res 或next。如果您确实将它传递到那里，您最终可能会遇到其他问题，因为这两个函数都调用res.status(status).send()。当这些函数没有 await 执行任何操作时，您还可以在这些函数上使用一些 async 关键字——这并没有破坏任何内容，只是浪费了几个字符:)。

【参考方案2】：

你打电话：

fileExists(req);

但是您对fileExists() 的实现希望您通过它(res, res, next) 而您没有通过它。因此，当fileExists() 尝试调用next() 时，您会收到错误next() is not a function。

我不清楚您到底要做什么，但也许您可以更改为将(req, res, callback) 传递给fileExists()，然后在该回调中继续处理。这就是您自己手动调用中间件的方式，看起来您正在尝试这样做：

const isAuthenticated = (req, res, next) => 
    const accessToken = valueOrNull((req.cookies && req.cookies['x-access-token']) || (req.headers && req.headers['x-access-token']));
    
    if (!accessToken) 
        console.log('accessToken not found')
        return res.status(401).send('UNAUTHORIZED');
    

    try 
        fileExists(req, res, () => 
            const  user  = verify(accessToken, ACCESS_SECRET);

            if (user) 
                console.log('VALID USER');
            
            return next();

        );
     catch (err) 
        console.log(err)
        return res.status(401).send('UNAUTHORIZED');
    

;