Node/Express重写Response.end方法追加功能报错如何解决

我和团队尝试在Express中间件中修改response.end方法，实现服务端向客户端返回响应前自动执行额外逻辑，初始实现代码如下：

return (req: Request, res: Response, next: NextFunction): NextFunction => {
    // reassign res.end in order to allow logger functionality before
    // a response is sent back the client
    const temp = res.end;
    
    res.end = () => {
        // instantiates PostQuery object with passed in query data from limiter middleware
        const postQuery = new PostQuery(gateURI, projectID, res.locals.graphqlGate);

        // our logger middleware functionality
        try {
            await postQuery.post();
        } catch (err) {
            if (err) console.log(err);
        }

        // our temp variable holding node's res.end definition
        return temp.call(this);
    };

    return next();
};

将该函数加入中间件链后，测试服务器抛出如下错误：

TypeError: Cannot read properties of undefined (reading 'finished')
    at end (node:_http_outgoing:856:19)
    at /Users/jon/Documents/Solo Projects/OSP/graphQL-gate-logger/src/index.ts:65:25

其中index.ts:65指向代码中return temp.call(this)一行。我们还尝试过直接调用return temp()、将temp绑定到res对象等方案，所有尝试均触发相同错误，想确认是否有其他方案可以实现需求，还是需要重新设计实现思路。

问题核心来自三个实现疏漏：

• 箭头函数的this绑定错误：用箭头函数定义覆写后的res.end时，箭头函数没有自身的this上下文，会直接继承外层中间件函数的this，根本不是当前响应实例res。原生res.end内部需要访问实例上的finished等属性，传入错误上下文自然会触发「读取undefined属性」的报错，之前尝试的绑定操作大概率因为箭头函数的上下文优先级规则没有生效。
• 参数透传缺失：原生res.end支持接收响应体片段、编码、执行回调三个入参，覆写的方法既没有接收也没有透传这些参数，哪怕上下文修复成功，也会出现响应内容丢失、编码异常等兼容问题。
• 异步函数标记缺失：覆写方法内使用了await关键字，但没有将函数标记为async，本身就存在语法执行异常。

方案1：修正覆写逻辑（适合需要在响应发送前修改内容的场景） #

不要用箭头函数定义覆写方法，提前绑定原始end方法的上下文为res，透传所有入参即可修复问题，参考代码：

return (req: Request, res: Response, next: NextFunction) => {
    // 提前绑定原始方法的上下文，避免后续执行时上下文丢失
    const originalEnd = res.end.bind(res);
    
    // 用普通函数定义，不要用箭头函数，同时接收所有入参做透传
    res.end = async function (...args: Parameters<Response['end']>) {
        const postQuery = new PostQuery(gateURI, projectID, res.locals.graphqlGate);
        try {
            await postQuery.post();
        } catch (err) {
            console.log('日志上报失败:', err);
        }
        // 透传所有参数调用原始end方法
        return originalEnd(...args);
    };

    next();
};

注意：如果保留await关键字，会阻塞响应流程——必须等异步逻辑（比如日志上报）执行完成才会把响应发给客户端，如果异步逻辑耗时长会直接拖慢接口响应速度。如果不需要等异步逻辑执行完再返回，直接去掉await，让异步任务后台执行即可。

方案2：监听标准响应事件（推荐，适合日志、统计类后置逻辑场景） #

Express的Response对象继承自Node.js原生http.ServerResponse，是标准的事件触发器，不需要手动覆写原生方法，直接监听finish事件即可。该事件会在响应完全发送给客户端后触发，不会阻塞响应，也不会出现覆写原生方法带来的兼容问题：

return (req: Request, res: Response, next: NextFunction) => {
    // 监听响应完成事件
    res.on('finish', () => {
        // 此处可正常访问req、res、res.locals上的所有数据
        const postQuery = new PostQuery(gateURI, projectID, res.locals.graphqlGate);
        // 不需要加await，响应已经发送给客户端，不影响用户侧体验
        postQuery.post().catch(err => {
            console.log('日志上报失败:', err);
        });
    });

    next();
};

如果需求是在响应发送前修改响应头、响应内容，选方案1即可；如果只是做日志上报、流量统计这类不需要修改响应的后置逻辑，优先选方案2，稳定性更高，不会和Express内置逻辑、第三方中间件产生冲突。

内容的提问来源于stack exchange，提问作者donjewey