【Express】解决Error: Can't set headers after they are sent.

版权声明:版权所有:CRPER(crper@outlook.com); 掘金|Github:CRPER; https://blog.csdn.net/bomess/article/details/52182538

问题

合并了小伙伴的分支,在跑开发模式的时候遇到标题那个错误,一脸蒙蔽的开始排查;

错误

这里写图片描述

  1. 看到这货,一开始的怀疑对象是模块没安装正确,我把node_modules的模块全部干掉,然后重新跑一遍npm install…浪费些许时间。依旧报错;

  2. 一脸懵逼,想到最近增加的东西【redis】;跑去看配置文件。。。。把redis模块的调用屏蔽了;

 // app.use(session({
    //     store: new RedisStore({
    //         "host": config.redis.host,
    //         "port": config.redis.port,
    //         "db": parseInt(config.redis.db),
    //         "pass": config.redis.pass ? config.redis.pass : null
    //     }),
    //     secret: 'jet',
    //     name: 'seed.sid',
    //     cookie: {maxAge:3600000,secure:false},
    //     resave: false,
    //     saveUninitialized: true
    // }));

这下子不会报未引用redis的问题;而这个错误依旧还在;

找了找。。终于发现罪魁祸首。。。回调函数少了return,导致信息一直在发送。。。

   //error function
   app.use(function (err, req, res, next) {
        res.status(err.status || 500).send({
            message: err.message,
            error: {},
            title: 'error'
        });
    });

解决

因为redis只有在注册和登录模块用到,我这边开发的功能暂时不需要这两个;

屏蔽了redis的模块的调用,且增加return就一切正常了

app.use(function (err, req, res, next) {
        return res.status(err.status || 500).send({
            message: err.message,
            error: {},
            title: 'error'
        });
    });

若是未屏蔽redis模块调用的情况下,该方法虽然增加了return,依旧会有一些其他信息回馈~~~

根本原因就是重复做出响应!!!导致了这些错误~~~

论代码的严谨的重要性,合并了分支最好排查一下;

展开阅读全文

没有更多推荐了,返回首页