Улучшить обработку ошибок проверки goose - PullRequest
Новая
       51

Улучшить обработку ошибок проверки goose

1 голос
/ 06 апреля 2020

У меня есть следующая схема с необходимыми проверками: 

var mongoose = require("mongoose");
var validator = require("validator");

var userSchema = new mongoose.Schema(
  {
    email: {
      type: String,
      required: [true, "Email is a required field"],
      trim: true,
      lowercase: true,
      unique: true,
      validate(value) {
        if (!validator.isEmail(value)) {
          throw new Error("Please enter a valid E-mail!");
        }
      },
    },
    password: {
      type: String,
      required: [true, "Password is a required field"],
      validate(value) {
        if (!validator.isLength(value, { min: 6, max: 1000 })) {
          throw Error("Length of the password should be between 6-1000");
        }

        if (value.toLowerCase().includes("password")) {
          throw Error(
            'The password should not contain the keyword "password"!'
          );
        }
      },
    },
  },
  {
    timestamps: true,
  }
);

var User = mongoose.model('User', userSchema);

Я передаю электронную почту и пароль через форму, отправляя запрос по почте по следующему маршруту: 

router.post("/user", async (req, res) => {
  try {
    var user = new User(req.body);
    await user.save();
    res.status(200).send(user);
  } catch (error) {
    res.status(400).send(error);
  }
});

module.exports = mongoose.model("User", user);

Всякий раз, когда я введите поле против правил проверки, я получаю очень длинное сообщение об ошибке, что очевидно. Но теперь я хочу улучшить обработку ошибок, чтобы их было легко интерпретировать пользователям. Вместо перенаправления на общую страницу ошибки c, как я могу перенаправить на ту же страницу регистрации и отобразить сообщения fla sh рядом с неправильными полями, сообщающими об ошибке? А также в случае успеха следует сделать нечто подобное, например, зеленое сообщение fla sh вверху.

Я использую e js для своих страниц регистрации.

Ответы [ 3 ]

1 голос
/ 06 апреля 2020

В блоке catch вы можете проверить, является ли ошибка проверкой mon goose, и динамически создать объект ошибки, подобный этому: 

router.post("/user", async (req, res) => {
  try {
    var user = new User(req.body);
    await user.save();
    res.status(200).send(user);
  } catch (error) {
    if (error.name === "ValidationError") {
      let errors = {};

      Object.keys(error.errors).forEach((key) => {
        errors[key] = error.errors[key].message;
      });

      return res.status(400).send(errors);
    }
    res.status(500).send("Something went wrong");
  }
});

Когда мы отправляем тело запроса следующим образом: 

{
   "email": "test",
   "password": "abc"
}

Ответ будет: 

{
    "email": "Please enter a valid E-mail!",
    "password": "Length of the password should be between 6-1000"
}
0 голосов
/ 06 апреля 2020

вы можете отправить 

res.status(400).send(error.message);

вместо: 

res.status(400).send(error);

, и вам также следует внести некоторые изменения в схему. 

validate(value){
            if (!validator.isEmail(value)) {
                throw new Error("Please enter a valid E-mail!");
            }
        }

с помощью 

validate: [validator.isEmail, "Please enter a valid E-mail!" ]

и для пароля: 

minlength: 6,
maxlength:1000,
validate:{
validator: function(el){
return el.toLowerCase() !== "password"
}
message: 'The password should not contain the keyword "password"!'
}
0 голосов
/ 06 апреля 2020

вы можете использовать validator, например, вместо того, чтобы выдавать ошибку: 

password:{
    type:String,
    required:[true, "Password is a required field"],

validate: {
  validator: validator.isLength(value,{min:6,max:1000}),
  message: "Length of the password should be between 6-1000"
}
    }
...