Błąd przekierowania ekspresowego: nie można ustawić nagłówków po ich wysłaniu.

Question

Kiedy ten kod trafi w linię przekierowania, zgłasza "Nie można ustawić nagłówków po ich błędzie" i nie przekierowuje. Jestem winny, że nie w pełni rozumiem nagłówki i jak działa ekspres z nimi. This link about this error trochę mnie dezorientuje, prawdopodobnie dlatego, że nie mam dość podstawowej wiedzy na temat tego, co się dzieje. Ponadto, wiem, że jest to trochę naiwne podejście do uwierzytelniania, ale staram się tylko, żeby podstawowe rzeczy działały.

app.post('/api/login', function(req, res) { 
    if (req.body.password === auth.password) { 
     auth.date = new Date() 
     res.redirect('/admin') 
     } else { 
     console.log("wrong pw") 
     } 
    }) 

UPDATE: dziękuję @Brendan Ashworth Tęskniłem oczywistym innego, który dodałem teraz i nie pojawia się błąd.

jednak ta linia nie zmienia zawartość mojej stronie

res.sendfile('./public/admin/views/tunes.html') 

to działało zanim owinął je ze sprawdzianem auth

var auth = require('../config/auth') 

module.exports = function(app) { 

/* 
* CONTENT API 
*/ 

//... 

/* 
* Admin Routes 
*/ 
app.get('/admin/login', function(req, res) { 
    res.sendfile('./public/admin/views/login.html') 
}) 

app.post('/api/login', function(req, res) { 
    if (req.body.password === auth.password) { 
     auth.date = new Date() 
     res.redirect('/admin') 
    } else { 
     res.json({message: 'Wrong password!'}) 
    } 
}) 

app.get('/admin', function(req, res) { 
    if (auth.date) { 
     res.sendfile('./public/admin/views/tunes.html') 
     console.log("test") // 
    } else { //added else 
     res.redirect('/admin/login') 
    } 
}) 

app.get('/admin/:url', function(req, res) { 
    if (auth.date) { 
     res.sendfile('./public/admin/views/' + req.params.url + '.html') 
    } else { //added else 
     res.redirect('/admin/login') 
    } 
}) 

// frontend routes 
// route to handle all angular requests 
app.get('*', function(req, res) { 
    res.sendfile('./public/views/index.html') 
}) 

}

FINAL UPDATE !! Ostatnią rzeczą, której potrzebowałem, było obsłużenie strony przekierowania po wysłaniu pliku. Proste uwierzytelnianie działa teraz idealnie!

$http.post('/api/login', $scope.auth).success(function() { 
     window.location.href = '/admin' 
    }) 

Answer 1

Wyjaśnienie błędu Can't set headers after they are sent error:

Wszystkie odpowiedzi HTTP śledzić tę podstawową strukturę:

.. Response Line .. 
.. Headers .. 

.. Body .. 

Jeśli chcesz przekierować użytkownika, najpierw Response Line zostanie wysłany z przekierowaniem kod (powiedzmy 300), następnie Headers zostanie wysłany z nagłówkiem Location: xxx.

Następnie możemy wreszcie wysłać ciało (nie w przypadku przekierowania, ale ogólnie). Jednak - w przypadku Twojego kodu - wysyłasz odpowiedź Body, a następnie próbując przekierować użytkownika. Ponieważ nagłówki (i linia odpowiedzi) zostały już wysłane (ponieważ wysłałeś ciało), nie może wysłać więcej nagłówków po ciele.

Przykładem tego w kodzie byłoby:

app.get('/admin', function(req, res) { 
    if (auth.date) { 
     res.sendfile('./public/admin/views/tunes.html') 
    } 
    res.redirect('/admin/login') 
}) 

Jeśli mam rację zakładając, rzeczywiście chcesz return po wywołaniu res.sendfile(). Jeśli auth.date jest zgodne z prawdą, to wyślesz plik (np. Odpowiedź ciała), a następnie podasz kod przekierowania - to nie zadziała.