Не удается установить заголовки после их отправки NodeJS Express - PullRequest
0 голосов
/ 28 сентября 2019

Я новичок в NodeJS, и когда я пытаюсь войти в систему с помощью электронной почты и пароля, я получаю следующую ошибку: (Часть ее)

Ошибка: Невозможно установить заголовки после их отправки.в validateHeader (_http_outgoing.js: 491: 11) в ServerResponse.setHeader (_http_outgoing.js: 498: 3) в ServerResponse.header (C: \ Users \ asuss \ Desktop \ server \ node_modules \ express \ lib \ response.js:771: 10) в ServerResponse.location (C: \ Users \ asuss \ Desktop \ server \ node_modules \ express \ lib \ response.js: 888: 15) в ServerResponse.redirect (C: \ Users \ asuss \ Desktop \ server \node_modules \ express \ lib \ response.js: 926: 18) в app.post (C: \ Users \ asuss \ Desktop \ server \ app.js: 114: 9) в Layer.handle [как handle_request] (C: \Пользователи \ asuss \ Desktop \ server \ node_modules \ express \ lib \ router \ layer.js: 95: 5) на следующем (C: \ Users \ asuss \ Desktop \ server \ node_modules \ express \ lib \ router \ route.js:137: 13)
в redirectHome (C: \ Users \ asuss \ Desktop \ server \ app.js: 42: 9) в Layer.handle [как handle_request] (C: \ Users \ asuss \ Desktop \ server \ node_modules\ express \ lib \ router \ layer.js: 95: 5) Ошибка: невозможно установить заголовки после их отправки.в validateHeader (_http_outgoing.js: 491: 11) в ServerResponse.setHeader (_http_outgoing.js: 498: 3) в ServerResponse.header (C: \ Users \ asuss \ Desktop \ server \ node_modules \ express \ lib \ response.js:771: 10) в ServerResponse.location (C: \ Users \ asuss \ Desktop \ server \ node_modules \ express \ lib \ response.js: 888: 15)

const express = require('express')
const session = require('express-session')
const bodyParser = require('body-parser')

const users = [
    {id: 1, name: 'Alex', email:'alex@gmail.com', password:"123"},
    {id: 2, name: 'Max', email:'max@gmail.com', password:"123"},
    {id: 3, name: 'Tiffany', email:'tiffany@gmail.com', password:"123"}
]

const app = express()

app.use(bodyParser.urlencoded({
    extended: true
}))

app.use(session({
    name: 'sid',
    resave: false,
    saveUninitialized: false,
    secret: "nikivekartal",
    cookie: {
        maxAge: 1000*60*60*2,
        sameSite: true,
    }
}))

const redirectLogin = (req, res, next) => {
    if(!req.session.userId){
        res.redirect('/login')
    }
    else {
        next()
    }
}

const redirectHome = (req, res, next) => {
    if(req.session.userId){
        res.redirect('/home')
    }
    else {
        next()
    }
}

app.get('/', (req, res) => {
    const {userId} = req.session;
    console.log("Session", req.session.userId)
    res.send(`
    <h1>Welcome!</h1>
    ${userId ? `  
    <a href='/home'>Home</a>
    <form method='post' action='/logout'>
        <button>Logout</button>
    </form>
    ` : `
    <a href='/login'>Login</a>
    <a href='/register'>Register</a>
    `}`)
})

app.get('/home', redirectLogin, (req, res) => {
    res.send(`
    <h1>Home</h1>
    <a href='/'>Main</a>
    <ul>
        <li>Apple</li>
        <li>Cherry</li>
    </ul>
    `)
})

// redirectHome mddleware ini eğer user authenticate olmuşsa login'e gitmesin
// home'a gitsin diye koyuyoruz
app.get('/login', redirectHome, (req, res) => {
    res.send(`
    <h1>Login</h1>
    <form method='post' action='/login'>
        <input type='email' name='email' placeholder='Email' required/>
        <input type='password' name='password' placeholder='Password' required/>
        <input type='submit' />
    </form>
    <a href='/register'>Register</a>
    `)
})

app.get('/register', redirectHome, (req, res) => {
    res.send(`

    <h1>Register</h1>
    <form method='post' action='/register'>
        <input name='name' placeholder='Name' required />
        <input type='email' name='email' placeholder='Email' required/>
        <input type='password' name='password' placeholder='Password' required/>
        <input type='submit' />
    </form>
    `)
})

app.post('/login', redirectHome, (req, res) => {
    const {email, password} = req.body

    if(email && password){
        const user = users.find(
            user => user.email === email && user.password === password
        )

        if(user){
            req.session.userId = user.id
            res.redirect('/home')
        }
    }

    res.redirect('/login')
})

app.post('/register', redirectHome, (req, res) => {

})

app.post('/logout', redirectLogin, (req, res) => {

})

app.listen(process.env.PORT || 3000, () => {
    console.log("Server is running");
})

Что мне делать и почему?Спасибо за внимание.

1 Ответ

0 голосов
/ 28 сентября 2019

Ваш маршрут /login может вызывать res.redirect() дважды.Добавьте возврат после одного из них, чтобы изменить это:

app.post('/login', redirectHome, (req, res) => {
    const {email, password} = req.body

    if(email && password){
        const user = users.find(
            user => user.email === email && user.password === password
        )

        if(user){
            req.session.userId = user.id
            res.redirect('/home')
        }
    }

    res.redirect('/login')
});

на это:

app.post('/login', redirectHome, (req, res) => {
    const {email, password} = req.body

    if(email && password){
        const user = users.find(
            user => user.email === email && user.password === password
        )

        if(user){
            req.session.userId = user.id
            res.redirect('/home')
            return;                   // <== add this return
        }
    }

    res.redirect('/login')
});
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...