Внутри MongoDB найти функцию, я вызываю и использую res.redirect, я получаю не могу установить ошибку заголовка? - PullRequest
Новая
       10

Внутри MongoDB найти функцию, я вызываю и использую res.redirect, я получаю не могу установить ошибку заголовка?

1 голос
/ 06 мая 2019

Я пытаюсь найти учетную запись пользователя / магазина в моей базе данных (mongoDB), а затем либо обновить страницу входа в систему, но с некоторыми ошибками, либо отправить их на нужную страницу.

Пока что отправка их на требуемую страницу работает, однако я получаю это сообщение об ошибке из консоли Node при попытке переадресации на страницу / signin. 

throw err;
      ^

Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client

Вот фактический код 

//Rendering Signin Page
app.get('/signin', function(req, res) {
    res.render('signin', {
        err: false,
    })
});

//Rendering Signin Page with err
app.get('/signin/err', function(req, res) {
    res.render('signin',{
        err: true,
    })
});


app.post('/signin', function(req, res) {
    let data  = req.body;
    User.find({email: data.email},'_id email password', function(err, docs) {
        if(err) {
            console.log(err);
        }
        else {


            //Finding the matching user
            for(i = 0; i < docs.length; i++) {
                if(data.password == docs[i].password) {
                    res.redirect('/'+docs[i]._id + '/userhome')
                }
            }

            if(docs.length === 0) {
                console.log('no user found')
                res.redirect('/signin/err');
                return;
            }
        }
    })
    Shop.find({email: data.email}, '_id email password', function(err,docs) {
        if(err) {
            console.log(err);
        }
        else {

            //Finding the matching user
            for(i = 0; i < docs.length; i++) {
                if(data.password == docs[i].password) {
                    res.redirect('/'+docs[i]._id + '/shophome')
                }
            }

            if(docs.length === 0) {
                console.log('no shop found')
                res.redirect('/signin/err')
                break;
            }
        }
    })
})

Также вот файл Pug, который я пытаюсь визуализировать (я не думаю, что это проблема) 

doctype html
html
    head    
        title uShop
        //Bootstrap CSS
        link(rel="stylesheet", href="https://stackpath.bootstrapcdn.com/bootstrap/4.3.1/css/bootstrap.min.css" integrity="sha384-ggOyR0iXCbMQv3Xipma34MD+dH/1fQ784/j6cY/iJTQUOhcWr7x9JvoRxT2MZw1T" crossorigin="anonymous")
        script(src='https://code.jquery.com/jquery-3.3.1.slim.min.js', integrity='sha384-q8i/X+965DzO0rT7abK41JStQIAqVgRVzpbzo5smXKp4YfRvH+8abtTE1Pi6jizo', crossorigin='anonymous')
        script(src='https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.14.7/umd/popper.min.js', integrity='sha384-UO2eT0CpHqdSJQ6hJty5KVphtPhzWj9WO1clHTMGa3JDZwrnQq4sF86dIHNDz0W1', crossorigin='anonymous')
        script(src='https://stackpath.bootstrapcdn.com/bootstrap/4.3.1/js/bootstrap.min.js', integrity='sha384-JjSmVgyd0p3pXB1rRibZUAYoIIy6OrQ6VrjIEaFf/nJGzIxFDsf4x0xIM+B07jRM', crossorigin='anonymous')
    body
        nav(class="navbar navbar-expand-md navbar-dark bg-dark sticky-top") 
            a(class="navbar-text" style="font-size:175%; color:white; padding:0px" href="/") uShop.com
        h1(class="display-2 text-center position-relative" style="top: 3rem; font-size: 400%") Sign In
        form(action="/signin" method="POST")
            div(class="form-row position-relative" style="top:7rem")
                label(class="col-lg-4 col-form-label position-relative text-right" for="inputEmail" style="font-size: 150%; top:-5px; left: -5px;") Email:
                input(class="form-control col-lg-4" type="email" name="email" id="inputEmail")
            //- if err == true
            //-     div(class="invalid-feedback") email incorrect
            br
            div(class="form-row position-relative" style="top:7rem")
                label(class="col-lg-4 col-form-label position-relative text-right" for="inputPassword" style="font-size: 150%; top:-5px; left: -5px;") Password:
                input(class="form-control col-lg-4" type="password" name="password" id="inputPassword")
            div(class="form-row position-relative" style="top:8rem")
                input(class="btn btn-primary btn-lg offset-lg-4 " type="submit" value="Sign In")

Ответы [ 2 ]

0 голосов
/ 06 мая 2019

Если вы хотите искать в коллекции user или shop.

  1. Добавьте async/await, это сделает ваш код более читабельным 
app.post('/signin', async function(req, res) {
....
})
Вам необходимо использовать findOne как для поиска в коллекции, так и для email и password в параметрах поиска, поэтому в коде 
const userResult = await User.findOne({ email: data.email, password: data.password }, '_id email password');
const shopResult = await Shop.findOne({ email: data.email, password: data.password }, '_id email password');
дополнительная проверка пароля не требуется As findOne возвращает object в результате, если email и password совпадают.Вы можете проверить, что если оба результата пусты, тогда перенаправьте, как показано ниже 
if(!userResult && !shopResult) {
  return res.redirect('/signin/err');
}

if(userResult) {
  return res.redirect('/'+userResult._id + '/userhome')
}

if(shopResult) {
  return res.redirect('/'+shopResult._id + '/shophome')
}

Добавление return при достижении redirect обеспечит завершение выполнения кода в этой точке.

Используйте try/catch для регистрации нежелательных ошибок.

MND-ссылка для async/await: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/async_function

0 голосов
/ 06 мая 2019 
app.post('/signin', function (req, res) {
    let data = req.body;
    User.find({ email: data.email }, '_id email password', function (err, docs) {
        res.redirect('/' + ...);
    })
    Shop.find({ email: data.email }, '_id email password', function (err, docs) {
        res.redirect('/' + ...);
    });
})

res.redirect не может быть запущен дважды в каждом запросе, но, по крайней мере, дважды ваш звонок вызывается

Как насчет этого? 

app.post('/signin', function (req, res) {

  let data = req.body;
    User.find({ email: data.email }, '_id email password', function (err, docs) {
        // res.redirect('/' + ...);

        Shop.find({ email: data.email }, '_id email password', function (err, docs) {
            res.redirect('/' + ...);
        });
    });
})

Кроме того, *

Вам нужно перенаправить на
'/' + docs[i]._id + '/shophome'
и
'/' + docs[0]._id + '/userhome' в signin API в один раз (один запрос)?

Я думаю, это невозможно при запросе http (rest api), и nodejs и другой язык такой же.

Кроме того, я думаю, что это цель normal user.

...