Quelle est l'erreur Mongoose Cast to ObjectId a échoué pour la valeur XXX au chemin «_id»?

Lors de l'envoi d'une demande à /customers/41224d776a326fb40f000001et un document avec _id 41224d776a326fb40f000001n'existe pas, docest nullet je retourne un 404:

  Controller.prototype.show = function(id, res) {
    this.model.findById(id, function(err, doc) {
      if (err) {
        throw err;
      }
      if (!doc) {
        res.send(404);
      }
      return res.send(doc);
    });
  };

Cependant, quand _idne correspond pas à ce que Mongoose attend comme "format" (je suppose) par exemple avec GET /customers/fooune erreur étrange est retourné:

CastError: échec de la conversion en ObjectId pour la valeur "foo" au chemin "_id".

Alors, quelle est cette erreur?

La findByIdméthode de Mongoose convertit le idparamètre en type de _idchamp du modèle afin qu'il puisse correctement rechercher le document correspondant. Il s'agit d'un ObjectId mais "foo"pas d'un ObjectId valide, donc le cast échoue.

Cela ne se produit pas 41224d776a326fb40f000001car cette chaîne est un ObjectId valide.

Une façon de résoudre ce problème consiste à ajouter une vérification avant votre findByIdappel pour voir si idun ObjectId est valide ou non:

if (id.match(/^[0-9a-fA-F]{24}$/)) {
  // Yes, it's a valid ObjectId, proceed with `findById` call.
}

Utilisez les fonctions existantes pour vérifier ObjectID.

var mongoose = require('mongoose');
mongoose.Types.ObjectId.isValid('your id here');

Analysez-vous cette chaîne comme ObjectId?

Ici, dans mon application, ce que je fais est:

ObjectId.fromString( myObjectIdString );

J'ai le même problème que j'ajoute
_id: String .in schema puis il commence à fonctionner

J'ai dû déplacer mes itinéraires par-dessus d'autres itinéraires qui captent les paramètres de l'itinéraire:

// require express and express router

const express = require("express");
const router = express.Router();

// move this `/post/like` route on top

router.put("/post/like", requireSignin, like);

// keep the route with route parameter `/:postId` below regular routes

router.get("/post/:postId", singlePost);

 if(mongoose.Types.ObjectId.isValid(userId.id)) {
        User.findById(userId.id,function (err, doc) {
            if(err) {
                reject(err);
            } else if(doc) {
                resolve({success:true,data:doc});
            } else {
                reject({success:false,data:"no data exist for this id"})

            }
        });
        } else {
            reject({success:"false",data:"Please provide correct id"});
        }

le mieux est de vérifier la validité

Dans mon cas, j'ai dû ajouter _id: Objectdans mon schéma, puis tout a bien fonctionné.

Vous pouvez également utiliser ObjectId.isValid comme suit:

if (!ObjectId.isValid(userId)) return Error({ status: 422 })

//Use following to check if the id is a valid ObjectId?

var valid = mongoose.Types.ObjectId.isValid(req.params.id);
if(valid)
{
  //process your code here
} else {
  //the id is not a valid ObjectId
}

J'ai été confronté à quelque chose de similaire récemment et je l'ai résolu en détectant l'erreur pour savoir s'il s'agissait d'une erreur Mongoose ObjectId.

app.get("/:userId", (req, res, next) => {
    try {
        // query and other code here
    } catch (err) {
        if (err.kind === "ObjectId") {
            return res.status(404).json({
                errors: [
                    {
                        msg: "User not found",
                        status: "404",
                    },
                ],
            });
        }
        next(err);
    }
});

Au 19 novembre 2019

Vous pouvez utiliser à isValidObjectId(id)partir de la version 5.7.12 de la mangouste

https://mongoosejs.com/docs/api/mongoose.html#mongoose_Mongoose-isValidObjectId

J'ai opté pour une adaptation de la solution @gustavohenke, implémentant le cast ObjectId dans un try-catch enroulé autour du code d'origine pour tirer parti de l'échec du casting ObjectId comme méthode de validation.

Controller.prototype.show = function(id, res) {
  try {
    var _id = mongoose.Types.ObjectId.fromString(id);



    // the original code stays the same, with _id instead of id:

    this.model.findById(_id, function(err, doc) {
      if (err) {
        throw err;
      }
      if (!doc) {
        res.send(404);
      }
      return res.send(doc);
    });



  } catch (err) {
    res.json(404, err);
  }
};

C'est une vieille question mais vous pouvez également utiliser le package express-validator pour vérifier les paramètres de la demande

express-validator version 4 (dernière):

validator = require('express-validator/check');

app.get('/show/:id', [

    validator.param('id').isMongoId().trim()

], function(req, res) {

    // validation result
    var errors = validator.validationResult(req);

    // check if there are errors
    if ( !errors.isEmpty() ) {
        return res.send('404');
    }

    // else 
    model.findById(req.params.id, function(err, doc) { 
        return res.send(doc);
    });

});

express-validator version 3:

var expressValidator = require('express-validator');
app.use(expressValidator(middlewareOptions));

app.get('/show/:id', function(req, res, next) {

    req.checkParams('id').isMongoId();

    // validation result
    req.getValidationResult().then(function(result) {

        // check if there are errors
        if ( !result.isEmpty() ) {
            return res.send('404');
        }

        // else
        model.findById(req.params.id, function(err, doc) {
            return res.send(doc);
        });

    });

});

Utilisez toujours mongoose.Types.ObjectId('your id')des conditions dans votre requête, il validera le champ id avant d'exécuter votre requête, par conséquent, votre application ne plantera pas.

OU vous pouvez le faire

var ObjectId = require('mongoose').Types.ObjectId; var objId = new ObjectId( (param.length < 12) ? "123456789012" : param );

comme mentionné ici, la méthode de recherche de Mongoose avec $ ou condition ne fonctionne pas correctement

La façon dont je résout ce problème transforme l'identifiant en une chaîne

je l'aime bien avec le backtick: `${id}`

cela devrait résoudre le problème sans frais généraux

ObjectId est composé des éléments suivants.

1. une valeur de 4 octets représentant les secondes depuis l'époque Unix
2. une valeur aléatoire de 5 octets (ID machine 3 octets et ID processeur 2 octets)
3. un compteur de 3 octets, commençant par une valeur aléatoire.

La méthode correcte pour valider si objectId est valide consiste à utiliser la méthode statique de la classe ObjectId elle-même.

mongoose.Types.ObjectId.isValid (sample_object_id)

Convertir la chaîne en ObjectId

import mongoose from "mongoose"; // ES6 or above
const mongoose = require('mongoose'); // ES5 or below

let userid = _id
console.log(mongoose.Types.ObjectId(userid)) //5c516fae4e6a1c1cfce18d77

Détection et correction de l'erreur ObjectID

Je suis tombé sur ce problème en essayant de supprimer un élément à l'aide de la mangouste et j'ai eu la même erreur. Après avoir examiné la chaîne de retour, j'ai trouvé qu'il y avait des espaces supplémentaires à l'intérieur de la chaîne retournée qui ont causé l'erreur pour moi. J'ai donc appliqué quelques-unes des réponses fournies ici pour détecter l'ID erroné, puis supprimer les espaces supplémentaires de la chaîne. Voici le code qui a fonctionné pour moi pour enfin résoudre le problème.

const mongoose = require("mongoose");
mongoose.set('useFindAndModify', false);  //was set due to DeprecationWarning: Mongoose: `findOneAndUpdate()` and `findOneAndDelete()` without the `useFindAndModify`



app.post("/delete", function(req, res){
  let checkedItem = req.body.deleteItem;
  if (!mongoose.Types.ObjectId.isValid(checkedItem)) {
    checkedItem = checkedItem.replace(/\s/g, '');
  }

  Item.findByIdAndRemove(checkedItem, function(err) {
    if (!err) {
      console.log("Successfully Deleted " + checkedItem);
        res.redirect("/");
      }
    });
});

Cela a fonctionné pour moi et je suppose que si d'autres éléments commencent à apparaître dans la chaîne de retour, ils peuvent être supprimés de la même manière.

J'espère que ça aide.

J'ai corrigé ce problème en modifiant l'ordre des itinéraires.

J'avais des problèmes avec cela et j'ai résolu de m'en mongoose.ObjectId(id)passerTypes