Was ist ein Mungofehler? Die Umwandlung in ObjectId ist für den Wert XXX im Pfad "_id" fehlgeschlagen?

Wenn das Senden einer Anfrage an /customers/41224d776a326fb40f000001und ein Dokument mit _id 41224d776a326fb40f000001nicht vorhanden ist, docist nullund ich ein 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);
    });
  };

Wenn _idjedoch nicht das übereinstimmt, was Mongoose als "Format" erwartet (nehme ich an), wird beispielsweise GET /customers/fooein seltsamer Fehler zurückgegeben:

CastError: Die Umwandlung in ObjectId ist für den Wert "foo" im Pfad "_id" fehlgeschlagen.

Was ist dieser Fehler?

Die findByIdMethode von Mongoose wandelt den idParameter in den Typ des _idModellfelds um, damit das passende Dokument ordnungsgemäß abgefragt werden kann. Dies ist eine ObjectId, aber "foo"keine gültige ObjectId, sodass die Umwandlung fehlschlägt.

Dies ist nicht der Fall, 41224d776a326fb40f000001da diese Zeichenfolge eine gültige ObjectId ist.

Eine Möglichkeit, dies zu beheben, besteht darin, vor dem findByIdAufruf eine Überprüfung hinzuzufügen, um idfestzustellen, ob es sich um eine gültige ObjectId handelt oder nicht:

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

Verwenden Sie vorhandene Funktionen zum Überprüfen der ObjectID.

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

Analysieren Sie diese Zeichenfolge als ObjectId?

Hier in meiner Bewerbung mache ich Folgendes:

ObjectId.fromString( myObjectIdString );

Ich habe das gleiche Problem, das ich
_id: String .in Schema hinzufüge, dann fängt es an zu arbeiten

Ich musste meine Routen über andere Routen verschieben, die die Routenparameter erfassen:

// 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"});
        }

Am besten überprüfen Sie die Gültigkeit

In meinem Fall musste ich _id: Objectmeinem Schema hinzufügen , und dann funktionierte alles einwandfrei.

Sie können ObjectId.isValid auch wie folgt verwenden:

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
}

Stand 19. November 2019

Sie können isValidObjectId(id)ab Mungo Version 5.7.12 verwenden

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

Ich habe mich für eine Anpassung der @gustavohenke-Lösung entschieden und Cast ObjectId in einem Try-Catch implementiert, der um den Originalcode gewickelt ist, um den Fehler des ObjectId-Castings als Validierungsmethode zu nutzen.

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);
  }
};

Dies ist eine alte Frage, aber Sie können auch das Express-Validator-Paket verwenden, um Anforderungsparameter zu überprüfen

Express-Validator Version 4 (neueste Version):

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);
        });

    });

});

Verwenden mongoose.Types.ObjectId('your id')Sie diese Option immer für Bedingungen in Ihrer Abfrage. Dadurch wird das ID-Feld überprüft, bevor Ihre Abfrage ausgeführt wird. Dadurch stürzt Ihre App nicht ab.

Ich war kürzlich mit etwas Ähnlichem konfrontiert und habe es gelöst, indem ich den Fehler abgefangen habe, um herauszufinden, ob es sich um einen Mongoose ObjectId-Fehler handelt.

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);
    }
});

ODER Sie können dies tun

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

Wie hier erwähnt, funktioniert Mongooses Suchmethode mit $ oder Bedingung nicht richtig

Die Art und Weise, wie ich dieses Problem behebe, besteht darin, die ID in einen String umzuwandeln

Ich mag es schick mit Backtick: `${id}`

Dies sollte das Problem ohne Overhead beheben

ObjectId besteht aus folgenden Dingen.

1. Ein 4-Byte-Wert, der die Sekunden seit der Unix-Epoche darstellt
2. ein 5-Byte-Zufallswert (Maschinen-ID 3 Bytes und Prozessor-ID 2 Bytes)
3. ein 3-Byte-Zähler, der mit einem zufälligen Wert beginnt.

Die korrekte Methode zum Überprüfen, ob die objectId gültig ist, besteht in der Verwendung der statischen Methode aus der ObjectId-Klasse.

mongoose.Types.ObjectId.isValid (sample_object_id)

Zeichenfolge in ObjectId umwandeln

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

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

Erkennen und Korrigieren des ObjectID-Fehlers

Ich bin auf dieses Problem gestoßen, als ich versucht habe, ein Element mit Mungo zu löschen, und habe denselben Fehler erhalten. Nachdem ich mir die Rückgabezeichenfolge angesehen hatte, stellte ich fest, dass in der zurückgegebenen Zeichenfolge einige zusätzliche Leerzeichen vorhanden waren, die den Fehler für mich verursachten. Daher habe ich einige der hier angegebenen Antworten angewendet, um die fehlerhafte ID zu erkennen und dann die zusätzlichen Leerzeichen aus der Zeichenfolge zu entfernen. Hier ist der Code, der für mich funktioniert hat, um das Problem endgültig zu beheben.

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("/");
      }
    });
});

Dies hat bei mir funktioniert und ich gehe davon aus, dass andere Elemente, die in der Rückgabezeichenfolge angezeigt werden, auf ähnliche Weise entfernt werden können.

Ich hoffe das hilft.

Ich habe dieses Problem behoben, indem ich die Reihenfolge der Routen geändert habe.