Express next() header error

Question

In a route I have this:

            if (elements.length <= 0) {
                var msg = 'no elements found';
                console.error(msg);
                var err = new Error('Not found');
                err.status = 404;
                err.message = msg;
                next(err);
            }

            console.log('Found ' + elements.length + ' elements');
            res.setHeader('Content-Type', 'application/json'); /*error*/
            res.status(200).json(elements);
            res.end();

The error handler that is defined last in app.js:

// development error handler
app.use(function(err, req, res, next) {
    res.type('application/json');
    res.status(err.status || 500);
    res.json({
        message: err.message,
        error: err
    });
});

I see that the error is sent as response in json. But I get this error on the line marked with /error/:

Can't set headers after they are sent.

Why is express returning from the error handler? I can see that it is continuing the execution of the route (from the console.log) Why is it continuing execution of the route?

Viktor Kireev · Accepted Answer · 2015-07-02 11:16:16Z

2

Inside if statement correct:

return next(err);

answered Jul 2, 2015 at 11:16

Viktor Kireev

1,2501 gold badge10 silver badges18 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Express next() header error

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related