Mongoose: ऑब्जेक्ट आईडी को सत्यापित करने का सबसे अच्छा तरीका

मुझे भी यही तकलीफ़ है। मैंने mongoose.Schema.ObjectId.isValid() के साथ-साथ mongoose.Types.ObjectId.isValid() की कोशिश की, लेकिन उन गुणों में से किसी में भी isValid विधि नहीं है। आखिर आपने इस समस्या का समाधान कैसे निकाला? मैं यह भी देखता हूं कि मोंगोडब में एक है और एक अन्य विकल्प के रूप में रेगेक्स भी है। मैं रेगेक्स का उपयोग नहीं करना पसंद करूंगा और न ही आवश्यकता होगी ('मोंगोडब')

यह मेरे लिए काम करता है:

var mongoose = आवश्यकता ('./node_modules/mongoose');
कंसोल.लॉग (mongoose.Types.ObjectId.isValid);
// [फ़ंक्शन: मान्य है]
console.log(mongoose.Types.ObjectId.isValid('53cb6b9b4f4ddef1ad47f943'));
// सच
console.log(mongoose.Types.ObjectId.isValid('bleurgh'));
// असत्य

संयोग से mongodb में विधि bson lib विधि है, जो केवल शून्य, 12 या 24 चार हेक्स स्ट्रिंग की जांच नहीं करती है - यह इस तरह एक रेगेक्स है:

var checkForHexRegExp = नया RegExp ("^ [0-9a-fA-F] {24}$");

तो यह बहुत हैकी नहीं हो सकता है अगर इसका उपयोग वहां किया जाता है

sValid () हमेशा सही लौटाता है यदि स्ट्रिंग में 12 अक्षर हैं

console.log(mongoose.Types.ObjectId.isValid("zzzzzzzzzzzz")); // सच

चूंकि "zzzzzzzzzzzz" तकनीकी रूप से एक वैध ऑब्जेक्ट आईडी है - ऑब्जेक्ट आईडी की एकमात्र परिभाषित विशेषता यह है कि इसकी 12 बाइट लंबी है। देखें mongodb/js-bson#112. लंबाई 12 की एक जेएस स्ट्रिंग में 12 बाइट्स हैं, मॉड्यूलो यूनिकोड अजीबताएं हैं। यदि आप लंबाई 24 हेक्स स्ट्रिंग की जांच करना चाहते हैं, तो बस स्ट्रिंग मिलान /^[a-fA-F0-9]{24}$/ . की जांच करें

"zzzzzzzzzzzz" मान्य ऑब्जेक्ट आईडी नहीं है। उदाहरण के लिए Mongo shell listiong (मोंगोडब संस्करण - 3.0.2):

> ObjectId('zzzzzzzzzzzz')
2015-04-29T18:05:20.705+0300 E QUERY    Error: invalid object id: length
    at (shell):1:1
> ObjectId('zzzzzzzzzzzzzzzzzzzzzzzz')
2015-04-29T18:06:09.773+0300 E QUERY    Error: invalid object id: not hex
    at (shell):1:1
> ObjectId('ffffffffffff')
2015-04-29T18:09:17.303+0300 E QUERY    Error: invalid object id: length
    at (shell):1:1
> ObjectId('ffffffffffffffffffffffff')
ObjectId("ffffffffffffffffffffffff")

क्योंकि मोंगोडब शेल का ऑब्जेक्टआईडी कंस्ट्रक्टर ऐसा लिखा गया है कि यह केवल हेक्स स्ट्रिंग्स को स्वीकार करता है। यह सुविधा के लिए मोंगो खोल में प्रतिबंध है, बीएसओएन प्रकार ऑब्जेक्ट आईडी के साथ नहीं। माना जाता है कि यह कुछ हद तक प्रतिकूल मामला है क्योंकि हेक्स स्ट्रिंग्स आमतौर पर ऑब्जेक्ट आईडी का प्रतिनिधित्व करती हैं, लेकिन अगर आपको यह पसंद नहीं है तो बस रेगेक्स /^[a-fA-F0-9]{24}$/ का उपयोग करें :)

जब हम किसी ऑब्जेक्ट आईडी पर isValid करने का प्रयास करते हैं, न कि स्ट्रिंग पर क्यों हम झूठा हो जाते हैं? क्या यह सच नहीं होना चाहिए और ObjectId एक वैध ObjectId है? इसका कोई मतलब नहीं है - शायद .toString() पर कॉल करें यदि यह एक वस्तु isValid पर पास की जा रही है?

@niftylettuce टिप्पणियों का #3365 पर स्वागत है। अभी हम बस bson पैकेज के ObjectId.isValid() फंक्शन को टाल देते हैं, जो mongoose में ObjectIds के बारे में लोगों के विचार से बिल्कुल मेल नहीं खाता। अगर आपको ऑब्जेक्ट आईडी दिया गया है, तो मैं सच लौटने के लिए एक पीआर खोलूंगा, यह पूरी तरह से उचित लगता है।

यहां एक पुराने मुद्दे पर वापस आ रहा है ... @atcwells समाधान mongoose.Types.ObjectId.isValid('53cb6b9b4f4ddef1ad47f943') मेरे लिए काफी अच्छा काम करता है, लेकिन यह अभी भी थोड़ा हैकी लगता है कि मेरे नियंत्रक को जांचना है कि कोई ऑब्जेक्ट आईडी मान्य है या नहीं - जब निश्चित रूप से यह एक बहुत ही सामान्य उपयोग का मामला है जो सर्वर पर एक खराब-निर्मित आईडी भेजने में सक्षम होना चाहता है और इसे क्रैश नहीं करना है।

आदर्श रूप से, यह कॉलबैक में err में बस कुछ लौटाएगा ताकि हम इसे सही तरीके से संभाल सकें और हमारे नियंत्रक का उपयोग करके सही HTTP स्थिति भेज सकें।

क्या कोई उपयोग-मामला है जहां यह कोर में उपयोगी कार्यक्षमता नहीं होगी ? यदि नहीं, तो शायद हम एक प्लगइन बना सकते हैं। मैंने एक त्वरित खोज की है और ऐसा कुछ भी प्रतीत नहीं होता है जो काम करता है - https://github.com/CampbellSoftwareSolutions/mongoose-id-validator यह सत्यापित करने के लिए है कि आईडी वास्तव में मौजूद है, जो कि ऐसा नहीं है हम यहां करना चाहते हैं - हम केवल यह सुनिश्चित करना चाहते हैं कि हम कोई अनकही त्रुटि उत्पन्न न करें।

अभी, मेरे एक्सप्रेस नियंत्रक में, हर बार जब मैं एक अनुरोध पर जाता हूं जिसमें ऑब्जेक्ट आईडी है, उदाहरण के लिए https://myproject/organisations/ {id} पर जाएं, मुझे कुछ ऐसा करना है:

if( !mongoose.Types.ObjectId.isValid(id) ){
    return res.sendStatus(400); // They didn't send an object ID
  }

... इससे पहले Organisation.findOne(); . करने जा रहे हैं

सुंदर बॉयलरप्लेट लगता है। मुझे एक प्लगइन या कुछ लिखने में खुशी हो रही है अगर कोई मुझे सही दिशा में इंगित कर सकता है कि कहां से शुरू करना है। एक प्लगइन की तरह प्रतीत नहीं होता क्योंकि यह वास्तव में एक स्कीमा चीज नहीं है ...

@shankiesan आपको ऐसा करने की आवश्यकता नहीं है, आईडी अमान्य होने पर नेवला क्वेरी वादे को अस्वीकार कर देगा।

var assert = require('assert');
var mongoose = require('mongoose');
var Schema = mongoose.Schema;

mongoose.connect('mongodb://localhost/test');
mongoose.set('debug', true);

var MyModel = mongoose.model('test', new Schema({ name: String }));

MyModel.findOne({ _id: 'invalid' }).exec().catch(error => console.error('error', error));

आउटपुट:

$ node gh-1959.js 
error { CastError: Cast to ObjectId failed for value "invalid" at path "_id"
    at MongooseError.CastError (/home/val/Workspace/10gen/troubleshoot-mongoose/node_modules/mongoose/lib/error/cast.js:19:11)
    at ObjectId.cast (/home/val/Workspace/10gen/troubleshoot-mongoose/node_modules/mongoose/lib/schema/objectid.js:147:13)
    at ObjectId.castForQuery (/home/val/Workspace/10gen/troubleshoot-mongoose/node_modules/mongoose/lib/schema/objectid.js:187:15)
    at cast (/home/val/Workspace/10gen/troubleshoot-mongoose/node_modules/mongoose/lib/cast.js:174:32)
    at Query.cast (/home/val/Workspace/10gen/troubleshoot-mongoose/node_modules/mongoose/lib/query.js:2563:10)
    at Query.findOne (/home/val/Workspace/10gen/troubleshoot-mongoose/node_modules/mongoose/lib/query.js:1239:10)
    at /home/val/Workspace/10gen/troubleshoot-mongoose/node_modules/mongoose/lib/query.js:2163:21
    at new Promise.ES6 (/home/val/Workspace/10gen/troubleshoot-mongoose/node_modules/mongoose/lib/promise.js:45:3)
    at Query.exec (/home/val/Workspace/10gen/troubleshoot-mongoose/node_modules/mongoose/lib/query.js:2156:10)
    at Object.<anonymous> (/home/val/Workspace/10gen/troubleshoot-mongoose/gh-1959.js:10:37)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.runMain (module.js:604:10)
  message: 'Cast to ObjectId failed for value "invalid" at path "_id"',
  name: 'CastError',
  kind: 'ObjectId',
  value: 'invalid',
  path: '_id',
  reason: undefined }
^C
$ 

इसके अलावा, पुन: प्लगइन्स http://thecodebarbarian.com/2015/03/06/guide-to-mongoose-plugins

आह मैं क्या कठपुतली रहा हूँ! बेशक, अस्वीकार किए जा रहे वादे को संभालो ... तर्क, मेरे दिमाग। धन्यवाद @vkarpov15

कॉलबैक भी काम करते हैं यदि वादे बहुत अधिक सिरदर्द हैं MyModel.findOne({ _id: 'invalid' }).exec(error => console.error('error', error)); :)

ObjectId.isValid() उपयोग न करें यदि यह उपरोक्त 12 बाइट्स से पहले से स्पष्ट नहीं है तो वैलेरी ने कहा। बस इसके द्वारा बहुत अच्छा जल गया:
ObjectId.isValid('The Flagship') === true

@atcwells अगर आप उस बिट को शामिल करने के लिए अपनी अत्यधिक उत्थान वाली टिप्पणी को अपडेट कर सकते हैं तो मुझे लगता है कि अन्य लोग इसकी सराहना कर सकते हैं क्योंकि मैं शुरुआत में आपके द्वारा कही गई बातों के आधार पर कर रहा था: ObjectId.isValid('The Flagship') === true

मजेदार हिस्सा है:
निम्नलिखित कथन सत्य लौट रहा है
mongoose.Types.ObjectId.isValid("दक्षिण अफ्रीका")

मैं किसके साथ चल रहा हूं (अभी के लिए) वादा पकड़ने में त्रुटि प्रकार की जांच कर रहा है। यदि यह 'ऑब्जेक्टआईड' है तो मुझे 404 लौटाया जाता है। मैं 404 लौटा रहा हूं क्योंकि एपीआई/वेब सेवा के उपभोक्ता को संसाधन नहीं मिला था या मौजूद नहीं था।

उदाहरण देखें:

  Widget.findByIdAndRemove(resourceId)
    .then((result) => {
      if (!result) {
        let error = new Error('Resource not found');
        error.status = 404;
        next(error);
      } else {
        res.redirect(303, '/');
      }
    })
    .catch((error) => {
      if (error.kind === 'ObjectId') {
        let error = new Error('Resource not found');
        error.status = 404;

        next(error);
      } else {
        next(error);
      }
    });

अपडेट करें:
नियंत्रक में प्रत्येक रूट हैंडलर में इसे जोड़ने के बजाय। मैं सभी हैंडलर को ग्लोबल कैच में जोड़ रहा हूं।

उदाहरण देखें:

app.use(function(err, req, res, next) {
  // set locals, only providing error in development
  res.locals.message = err.message;
  res.locals.error = req.app.get('env') === 'development' ? err : {};

  if (err.kind === 'ObjectId') {
    err.status = 404;
  }

  // render the error page
  res.status(err.status || 500);
  res.render('error');
});

क्या यह स्कीमा में मान्य संभव है? एक सर्वोत्तम अभ्यास नहीं दोहराना है कि अगर, लेकिन मैं कास्ट त्रुटि घटना लॉग करना चाहता हूं।

मोंगोज़ रेगेक्स /^[a-fA-F0-9]{24}$/ को isValid के लिए लागू क्यों नहीं कर सकता है जब यह केवल MongoDB से निपट रहा है। यह बहुत भ्रमित करने वाला है। हम इस मुद्दे को डिबग करने में सिर्फ एक घंटा बिताते हैं और यह इतनी मूर्खतापूर्ण बात निकली कि आप कभी नोटिस नहीं करेंगे

मैं इस npm पैकेज https://www.npmjs.com/package/valid-objectid का उपयोग करने की सलाह दूंगा
यह पूरी तरह से काम करता है

मैंने इसके लिए वर्कअराउंड बनाया है:

function isObjectId(value) {
  try {
      const { ObjectId } = mongoose.Types;
      const asString = value.toString(); // value is either ObjectId or string or anything
      const asObjectId = new ObjectId(asString);
      const asStringifiedObjectId = asObjectId.toString();
      return asString === asStringifiedObjectId;
    } catch (error) {
      return false;
    }
}

@mstmustisnt मुझे लगता है कि कोशिश/पकड़ने की जरूरत है, अगर मान "123" है तो यह एक त्रुटि फेंक देगा।

अगर यह किसी की मदद करता है, तो मैं ऊपर दिए गए ObjectId.isValid() दृष्टिकोण का अनुकूलन कर रहा हूं। क्योंकि मेरे उपयोग के मामले में, मैं अपनी आईडी या उसके यूआरएल-स्लग द्वारा संसाधन प्राप्त करने में सक्षम होना चाहता हूं, उदाहरण के लिए:

GET /users/59f5e7257a92d900168ce49a ... या ...
GET /users/andrew-shankie

... मैंने पाया है कि यह मेरे नियंत्रक में अच्छी तरह से काम करता है:

  const { id } = req.params;

  const query = {
    $or: [{ slug: id }],
  };

  if (mongoose.Types.ObjectId.isValid(id)) query.$or.push({ _id: id });

  User.findOne(query)
    .exec((err, user) => { ... }

इस मामले में, एक 12-बाइट स्ट्रिंग अभी भी एक वैध ऑब्जेक्ट आईडी है, इसकी खोज करने से त्रुटि को फेंकने के बजाय केवल शून्य-लंबाई वाली सरणी मिलती है। और क्योंकि मैं एक $or क्वेरी का उपयोग कर रहा हूं, यह तब URL स्लग (मेरा अन्य संभावित विकल्प) द्वारा खोज करता है।

सबसे सुंदर समाधान नहीं हो सकता है, लेकिन यह मेरे लिए काम करता है।

@victorbadila हाँ, बिल्कुल। मैंने अभी एक संकेत दिया है। मेरी टिप्पणी संपादित, वास्तव में केवल उस कोड को रखता है जिसका मैं वास्तव में उपयोग करता हूं।

Validator.js में एक अंतर्निहित isMongoId विधि है

जब भी मैं नेवला का उपयोग करता हूं, मैं हमेशा इसे कुछ स्थिर सहायक विधियों के साथ बढ़ाता हूं:

const mongoose = require('mongoose');
const {Types: {ObjectId}} = mongoose;

//Helper to check if an ID is an object ID
mongoose.isObjectId = function(id) {
  return (id instanceof ObjectId);
};

//Helper to validate a string as object ID
mongoose.isValidObjectId = function(str) {
  if (typeof str !== 'string') {
    return false;
  }
  return str.match(/^[a-f\d]{24}$/i);
};

आप इसे अपने डेटाबेस इनिशियलाइज़ेशन स्क्रिप्ट के एक भाग के रूप में चला सकते हैं, इसलिए विधियाँ आपके ऐप में हमेशा उपलब्ध रहती हैं।

यदि ObjectId.isValid(id) सत्य है, तो हम (new ObjectId(id).toString()) के मान और id को आंक सकते हैं।

const mongoose = require('mongoose');
const {Types: {ObjectId}} = mongoose;
const validateObjectId = (id) => ObjectId.isValid(id) && (new ObjectId(id)).toString() === id;