node.jsのexpress-validatorで検証が出来ない。

Question

expressでexpress-validatorモジュールを使ってパスワードとパスワード（確認）の検証しようとしていますが、下記コードが効かず、model.save以下に移ってログインしてしまいます。（コードはドキュメントを参考して書いています）

body('passwordConfirmation').custom((value, req) => {
            if (value !== req.body.password) {
            throw new Error('パスワード確認がパスワードと一致しません');
            }
            return true;
        }),
model.save(function(err){
            if (err) {
                res.status(403).send('error:'+err);
            } else {
                res.render('home', {title: 'Laravel', user: req.body.name});
            }
        });

body('passwordConfirmation')をconsole.logで出力しましたが、html側のformタグname属性の入力値が入っているようではないようです。（よく分かっていないのでformのname名とmodel側のkey名をpasswordConfirmationに統一しています）
他にbody('password')を出力しても同じような内容で、このbodyは何を表しているのか分かりません。
しかしbody('password').isLength({ min: 7 })はしっかり効いてくれてます。
なのに上記のコードが効かないのはなぜなのか？
どうやってデバッグ＆検証すればよいのか教えて下さい。

宜しくおねがいします。
【修正後】
上記のコードの全文（controller.js）とview/login.ejsを追加します。

const mongoose = require('mongoose');
const { body, validationResult } = require('express-validator');
const schema = require('../models/model');
const Model = mongoose.model('Model', schema);

exports.login = function(req, res){
    res.render('login', {title: 'Laravel'});
};
exports.loginPost = 
    function(req, res){
        const errors = validationResult(req);
        const model = new Model();
        console.log(body);
        if (!errors.isEmpty()) {
            return res.status(400).json({ errors: errors.array() });
        };
        body('passwordConfirmation').custom((value, req) => {
            //console.log(req.body.password);
            console.log(body.passwordConfirmation);
            if (value !== req.body.password) {
            throw new Error('パスワード確認がパスワードと一致しません');
            }
            return true;
        }),
        model.save(function(err){
            if (err) {
                res.status(403).send('error:'+err);
            } else {
                res.render('home', {title: 'Laravel', user: req.body.name});
            }
        });
    }

<%- include('partials/header') %>
<div id="big-container">
    <div id="login-first-container"><%= title %></div>
    <div id="login-second-container">
        <form action="#" method="post">
            <p>name<input type="text" name="name"></p>
            <p>Password<input type="text" name="password"></p>
            <p>Password（確認）<input type="text" name="passwordConfirmation"></p>
            <button name="login">Login</button>
            <a href="#">Forget Your Password?</a>
        </form>

    </div>
</div>
<%-include('partials/footer') %>

Comments

[suama]

一部だけだと思いますが、できるだけ全部は載せていたくのは可能でしょうか？
あと、送信するデータはPOSTメソッドだと思いますが、フォーム側のHTMLもあると良さそうな気がします。

[hokosugi]

了解しました。修正します。

[suama]

ありがとうございます！
先に思いついたところを書いてみますので、よかったら確認してみてくださいね！

Answer 1

こんにちは。
わたしもちょっとしか使っていないので詳しくないのですが、body('passwordConfirmation').custom() でまず先にバリデーションを済ませたあと、(req, res) => {...}の処理をするので、modal.saveもこの部分で差し込まないといけないのではと思います。

通常のリクエスト処理は (req, res) => ... のところで行いますが、express-validatorはその前にチェーンの形でバリデーション処理を挟みます。

const { body } = require('express-validator');

app.post(
　# エンドポイントはソースコードが一部だけだったので適当です。
  '/user',
  body('passwordConfirmation').custom((value, { req }) => {
    if (value !== req.body.password) {
      throw new Error('Password confirmation does not match password');
    }

    // Indicates the success of this synchronous custom validator
    return true;
  }),
  (req, res) => {
    // バリデーション処理をして、結果を判定
　　const errors = validationResult(req);
   // バリデーションエラーがあれば先に4xxを返す
　　if (!errors.isEmpty()) {
      res.status(403).send(`error: $errors`);
   }
    
    // modal.save はこの後に続ける
    model.save(function(err){ ..... }) 
  },
);

body('password').isLength({ min: 7 }) はしっかり効いてくれてます。

ここは、Getting Startedのところの例でしょうか？
https://express-validator.github.io/docs/index.html

上記に添えた通り、まず先にバリデーション（複数OK) を済ませたあと、(req, res) => {...}の処理をするので、この例の通りに書かれていましたら、確かにちゃんと処理されると思います。

Comments

[hokosugi]

ありがとうございます！上手くいきました。`exports.loginPost`をfunctionから始めないとエラーとなるので`router`側に`body('passwordConfirmation').custom`を移してfunctionから出したら機能してエラー表示が出るようになりました。コード自体に問題はなく、router側でcontroller.jsを呼び出す際にエラーが出るのでfunction内に`body('passwordConfirmation`を格納したことが原因でした。改めてお礼申し上げます。

[suama]

良かったです！
結果的にはご自身で解決されたのでは、と思いますので恐縮です。。。
コードの記載もありがとうございます。

[hokosugi]

いえいえ、ご回答がなかったら出来ておりませんでした。改めてお礼申し上げます。