ExpressJS "Fehler: Kann Header nicht festlegen, nachdem sie gesendet werden."

Question

Ich stoße also auf ein Problem mit ExpressJS und finde keine Dokumentation, um das Problem zu lösen.

Tech:

• Körper-Parser: 1.17.0
• exprimieren 4.15.0
• multer: 1.3.0
• MongoDB
• Postman

Ansicht ist derzeit 3 Felder:

• Name (erforderlich)
• Tagline (erforderlich)
• Bild (optional)

Was ich versuche zu tun, Fehler Handle auf das Bild, bevor Sie etwas schreiben in die Datenbank. Das Bild kann nur von mime typeimage/jpeg oder image/png sein, um zu verhindern, dass HTML mit schädlichen JS hochgeladen wird.

Ich glaube, das Problem scheint zu sein, dass ich nicht richtig einen Fehler Auslösung während Sie durch die Bildprüfung conditionals laufen und mehrere Antworten zu senden, die die Error: Can't set headers after they are sent.

Aufrechnungs

drinks.routes.js

var express = require('express'); 
var router = express.Router(); 
var jwt = require('jsonwebtoken'); 
var multer = require('multer'); 
var passport = require('passport'); 
var config = require('../config/main'); 
var upload = multer({ dest: 'uploads/images' }) 

var Drink = require('../models/drinks.model.js'); 

router.use(function(req, res, next){ 
    next(); 
}); 

... 

.post(passport.authenticate('jwt', { session: false }), upload.single('image'), function(err, req, res, next){ 
    var drink = req.body; 
    var drinkImage = req.file; 
    if(typeof drinkImage !== "undefined"){ 
    console.log('image was uploaded'); 
    if(drinkImage.mimetype !== "image/jpeg" || drinkImage.mimetype !== "image/png"){ 
     console.log('Image was not a JPEG or PNG', drinkImage.mimetype); 
     res.status(500).send({ error: "Your image was incorrect"}); // >>>>>>>>>>>>>>> The error seems to be coming from here. Unsure of how to properly raise a flag to tell the response to the client. Have tried res.send(), the res.status().send(), res.json(), currently working with next() method to keep going on but not sure how to define err if that is the case 
    } 
    console.log('image correct mimetype'); 
    } else { 
    drinkImage = {}; // Setting this as an empty object so it doesn't throw an error with the model which is looking for `image: drinkImage.name` 
    } 
    Drink.createDrink(drink, drinkImage, function(err, drink, drinkImage){ 
    if(err){ 
     console.log('Error adding Drink', err); 
     res.send(err); 
    } 
    res.status(200).json(drink) 
    }); 
}); 

Thema Forschung

• Express Error Handling
• SO Can't Set Headers Twice

Answer 1

Dieses Problem ist Javascript die asynchrone Natur zurückzuführen.

Der Code sollte Drink.createDrink() nach der Antwort error 500 nicht ausführen.

if (drinkImage.mimetype !== "image/jpeg" || drinkImage.mimetype !== "image/png") { 
    console.log('Image was not a JPEG or PNG', drinkImage.mimetype); 
    res.status(500).send({ error: "Your image was incorrect"}); 
    return; // THIS IS VERY IMPORTANT! 
}