難しさを扱うjavascriptの約束

Question

こんにちは誰かが私のコードを見れますか？私はバックエンドで認証を実装しています。JavaScriptの理解を深めるためのサイドプロジェクトとして構築しています。

なぜ私はmongoose findからの約束が未定義を返すのか分かりません。

私には2つの機能があります。 1つはコントローラに渡されるヘルプ機能です。

import db from '../models'; 
import bcrypt from 'bcrypt'; 
import validateLogin from '../validations/login'; 
import isEmpty from 'lodash/isEmpty'; 
const loginController = {}; 

loginController.login = function(req,res){ 
    validateLogin(req.body).then(({isValid, errors }) => { 
    // isValid is undefined here 
    // This is the problem 
     if(isValid){ 
      // give token 
      res.status(200).json({ 
       success: true, 
      token: 'here is your token' 
      }); 
     } else { 
      res.status(401).json({ 
      errors 
      });  
     }  
    }).catch(err =>{ 
     console.log(err) 
    });  
}; 

export default loginController; 

もう1つは、ヘルパー関数が有効または無効の応答を返すかどうかによってトークンを使用するコントローラ機能です。

import validator from 'validator'; 
    import db from '../models'; 
    import isEmpty from 'lodash/isEmpty'; 
    import bcrypt from 'bcrypt'; 

    function validateLogin(data){ 
     const { userInput, password } = data; 
     const errors = {}; 
     if(validator.isEmpty(userInput)){ 
     errors.userInput = 'username is required'; 
     } 
     if(validator.isEmpty(password)){ 
     errors.password = 'password is required'; 
     } 
     return db.User.find({$or:[{ username: userInput }, { email: userInput }]}).then(existingUser =>{ 
     if(existingUser.length > 0){ 
      // User exists, check if password matches hash 
      const user = existingUser[0]; 
      bcrypt.compare(password, user.password_digest).then(valid => { 
       if(!valid){ 
        errors.password = 'Invalid Password'; 
       } 
       console.log('from prmomise'); 
       return { 
        isValid: isEmpty(errors), 
        errors 
       }; 
      }).catch(err => console.log(err)); 
      } else { 
      errors.userInput = 'username or email does not exist'; 
      return { 
       isValid: isEmpty(errors), 
       errors 
      }; 
      } 
     }); 
    } 

    export default validateLogin 

Answer 1

returnがbcryptコールのために欠けているように見えます。それがなければ

return bcrypt.compare(password, user.password_digest).then(valid => { 
    // password check and returning result object 
}).catch(err => console.log(err)); 

あなたvalidateLogin機能が最終的に解決さundefined約束を返します。