Mongoose: モデル更新メソッドはバリデーターを尊重しません

strictスキーマ設定を有効にして、無効なスキーマパスを無視し、残りを更新することをお勧めします。 エラーはスローされません。

new Schema（{..}、{strict：true}）

スキーマに存在しないパスを設定するとスローされるが、 model.update間にスローされない、マージを待機している同様のプルリクエストがあります。 たぶん、それを上記のPRに追加して、エラーをコールバックに渡す必要があります。

こんにちは、迅速な返信ありがとうございます。

私は無効なパスを参照していませんでしたが、特定のパス内の制約を参照していました。

この例を見てください

var User = new Schema（{
ユーザー名：{
タイプ：文字列、
小文字：true、
トリム：true、
ユニーク：true
}、
名前： {
タイプ：文字列、
トリム：true、
必須：true、
一意：false、
インデックス：true
}、
パスワード：{
タイプ：文字列、
必須：true、
選択：false、
セット：暗号化
}、
役割： {
タイプ：文字列、
"デフォルト"： 'ユーザー'、
"enum"：['user'、 'admin'、 'root']
}、
作成した： {
タイプ：日付、
「デフォルト」：Date.now
}
}、{strict：true}）;

users.update（{"username"： "test"}、{"role"： "thisShowldFail"}、function（err、val）{
..。
}）;

厳密なスキーマオプションが有効になっている場合でも、「test」ユーザーには「thisShowldFail」の役割があることに注意してください。
リスペクト列挙型や他の同様のオプションを更新するべきではありませんか？

あなたは正しいです、それは列挙型を検証する必要があります。 どこかにチケットがあると思いました。

+1、この問題のために#updateを使用できません。

.updateに、新しいドキュメント+ .saveが実行するすべてのことを実行させることは可能ですか？

• デフォルト
• セッター
• カスタム検証
• 列挙型

アップサートオプションが使用されている場合も

• 必要

現在はありません

つまり、実装は可能/計画されていますか...？ :)

それはいいかもしれません。 関連＃472

+1

更新で使用されないのは完全に直感的ではありません：デフォルト、セッター、検証、列挙型。 コア操作がスキーマに準拠していない場合、スキーマはどのように使用されますか。

+1

私はマングースが大好きですが、それが更新を検証しない場合、それは私にとってはるかに役に立ちません。 大きなドキュメントがあり、検証のためだけにダウンロードしたくありません。

これは私が飛び込んでいることです。私はこれを実装しようとして、少し飛び込みます。
目標は次のことです。

• デフォルト
• セッター
• 検証
• 列挙型
• 必要

私が成功するかどうかさえわかりませんが、私はそれをやってみます。 バリデーターから始めます。

幸運を祈ります！

メルク。

+1

+1

.updateが検証をサポートしていれば、すばらしいと思います。

バンプ+1

でこぼこのバンプ+1

更新時に検証を追加してください！

+1

編集：-2マングースが検証を担当するべきではないと思うので..検証が必要な場合は、JSONスキーマ検証を許可するもののような他の特殊なライブラリの使用を検討する必要があります

@thalesfspはい。 これは過去に試みられましたが、ドキュメントがメモリに存在せず、いくつかの状況で壊れて動作に一貫性がなく混乱するため、ルールが非常に不安定になります。

オープンソースの美しさ：機能が必要な場合は、それを記述して、正しく機能することを証明するテストとドキュメントに合格したプルリクエストを送信できます。

かなり深刻な制限。 パッチを適用する場合、スキーマオブジェクトを直接クロールして、バリデーターを実行する必要があるかどうかを判断するのが現実的でしょうか。 私は主に、更新に適用されるカスタムバリデーターと列挙型ルールに関心があります。 他のスキーマ制約は、ドキュメントが保存されたときにすでに適用されているはずだと思います。 列挙型とバリデーターだけに至るまで問題をこのように単純化することは理にかなっており、更新中にドキュメントを用意する必要がなくなりますか？

+1バンプ、これは非常に便利な機能です-更新を行うとき、スキーマから最小/最大などが尊重されることを確認したい-そうでなければ、マングースができることのために多くの定型ロジックを実行しています

これが起こるのを見るのは素晴らしいことです。 現時点での回避策は、オブジェクトを見つけ、フィールドを変更してから保存することです。これにより、検証ミドルウェアが起動します。 ドキュメントによると：

Tank.findById(id, function (err, tank) {
  tank.size = 'large';
  tank.save(function (err) {
   // Document updated, do something with it
  });
});

updateコマンドはMongoに直接委任され、ドキュメント全体が検証のためにメモリに保存されないため、これは注意が必要です。 したがって、 @ BrianHoldsworthによって提案されたアプローチは、更新されるフィールドに対してのみ検証を実行するスキーマを解析することから始めるのに適した場所のようです。

@aheckmannは、このパッチを試みた人が同じ間違いを二度と起こさないように、以前の（失敗した）実装作業の詳細を教えていただけますか？

私は主に、更新に適用されるカスタムバリデーターと列挙型ルールに関心があります。 他のスキーマ制約は、ドキュメントが保存されたときにすでに適用されているはずだと思います。 列挙型とバリデーターだけに至るまで問題をこのように単純化することは理にかなっており、更新中にドキュメントを用意する必要がなくなりますか？

@BrianHoldsworthこれは単純化しrequired: true検証制約のあるフィールドが空の文字列に更新された場合はどうなりますか？ 検証エラーを発生させるには、これが必要になります。

私もこれに興味があります。 おそらく、.update（）メソッドをオーバーライドして検証を実行するプラグインを作成できますか？ そうすれば、部分的なソリューションでも実装できます。 一方、コアにある場合は、あらゆる種類の検証を処理し、100％堅牢であることが期待されます。

カスタムバリデーターと列挙型の両方で、これにも遭遇しました。 検索してから更新することは可能ですが、ドキュメントのサブドキュメント構造が異なる場合、一般的なケースコードを記述することは非常に困難です。

+1バンプこれ。

ブライアンのソリューションはかなりエレガントに見えます。 パッチがベータ版に達したときに通知を受け取りたいです。

+1

この+1が必要

+1

+1

+1

+1

3.9.3では、 update()にsetDefaultsOnInsertとrunValidators 2つの特別なオプションがあり、デフォルトを設定してクエリでバリデーターを実行します。 たとえば、テストを参照してください。実際のドキュメントはまだありません:(

どうもありがとうございました-これは素晴らしい修正です！

@ vkarpov15私はあなたのコードを使用して、

こんにちは、アップデート時に検証を使用するために、安定バージョン3.9がいつリリースされるか知っていますか？

ありがとう

@AlexandreAWE良い質問です。 私は現在、4.0と3.9.xブランチのベータテストで物事をまとめていますが、今は「完了したら完了します」ということです。 クリスマス前にRCを出したいです。

@ vkarpov15 3.9.xの安定版ではどうですか？ 現在のプロジェクトを毎回見つけて保存する代わりに、更新時に検証を行うのは素晴らしいことです。

@andrewholstedは、 mongodbドライバー2.0とmongodbサーバー2.8が安定するのを待っています。これは今月行われることを願っていますが、最新バージョンのmongodbと最新ドライバーをサポートしないと4.0を出荷することはできません。 詳細については、私のブログを参照し

私による潜在的にばかげた質問ですが、mongodbに渡す前に、更新されているデータを取得し、スキーマに対して検証してみませんか？

また、+ 1

マークして待つ

3.9が安定するまで待つことはできません:)

この機能はまだありますか？ 3.9を熱心に待っています。

npm install mongoose@unstableを使用して、不安定なバージョンを取得できます。 4.0の現在のETAは3月25日です-これを確認するのに適した場所はマイルストーンページです

昨夜この問題に遭遇しました。 私だけじゃなくてよかった！ この変更がリリースされることに興奮しています。ありがとうございました。

+1

この問題がクローズとマークされている理由がわかりません。 私はまだそれに直面しています。

検証ドキュメントの「

runValidatorsがtrueに設定されていても、これがfindOneAndUpdate（）の列挙型検証で機能するかどうかは実際にはわかりません。

@ m1cahは、あなたがやろうとしていることを示す例を提供してください。 これについてのテストがあり、合格しています...

@ vkarpov15これはそれを示す短い例だと思います： http ： //code.runnable.com/VYhGbVhereIYdbst/update-validation-enum-for-mongoose-and-databases

1つの問題は、上記の例が先史時代のマングースを使用していることです。

root<strong i="6">@runnable</strong>:~# head node_modules/mongoose/package.json                                                                                                                 
{                                                                                                                                                                         
  "name": "mongoose",                                                                                                                                                     
  "description": "Elegant MongoDB object modeling for Node.js",                                                                                                           
  "version": "3.6.14",                                                                                                                                                    
  "author": {                                                                                                                                                             
    "name": "Guillermo Rauch",                                                                                                                                            
    "email": "[email protected]"                                                                                                                                   
  },                                                                                                                                                                      
  "keywords": [                                                                                                                                                           
    "mongodb",  

4.xにアップグレードしてみてください。動作するはずです。

列挙型とfindOneAndUpdateメソッドで動作しますか？
マングース4.2.6では失敗するようですが、悪い値を設定できます。

スキーマ：

var UserSchema = new Schema({
    first_name: {
        type: String,
        required: true,
    },
    last_name: {
        type: String,
        required: true,
    },
    email: {
        type: String,
        unique: true,
        required: true,
    },
    embededData: [{
        type: {
            type: String,
            enum: ['value1', 'value2', 'value3']
            required: true
        }
    }]
}, { strict: true });

FindOneAndUpdateメソッド：

UserModel.findOneAndUpdate(
    {_id: uid}, 
    {$push: {embededData: data}}, 
    { runValidators: true }, function(err) {
});

次に、 embededData.type = 'Panda';プッシュできます

更新バリデーターのドキュメントと＃2933を参照してください-更新バリデーターは$pushでは実行されず、 $setと$unset

+1

私が見る限り、これは、配列であるフィールドでカスタムバリデーターを使用している場合にはまだ機能しません。

たとえば、次のコードスニペット：

var mongoose = require('mongoose');
mongoose.connect('mongodb://localhost/myTestDB');

var db = mongoose.connection;

db.on('error', function (err) {
console.log('connection error', err);
});
db.once('open', function () {
console.log('connected.');
});

var Schema = mongoose.Schema;
var userSchema = new Schema({
  _id : String,
  name : {
    type: [String],
    validate: {
        validator: function (str) {
            return str.length > 1
        }
    },
  }
});


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

User.findOneAndUpdate({"name": ["John", "Doe"]}, { 
  $setOnInsert: {
    name: ["John"],
  },
}, { runValidators: true, upsert: true, new: true }, function (err, data) {
  if (err) {
    return console.log(err);
  } else {
    // console.log(data.validateSync())
    return console.log('Updated', data);
  }
});

含まれているカスタムバリデーターが1以下の長さの名前配列を明示的に禁止している場合でも、エラーをスローせずに["John"]のnameフィールドを持つようにユーザーを更新できます。 console.log(data.validateSync())行のコメントを解除して検証を強制すると、実際には適切なエラーメッセージが返されることからわかるように、バリデーター自体は問題なく機能します。 問題は、 runValidators=trueオプションを含めているにもかかわらず、この検証がfindOneAndUpdate()呼び出し内で行われていないことです。

バグのようですが、別の問題を開くことができますか？

はい、 https：//github.com/Automattic/mongoose/issues/4039を開きました

ありがとう

上記の問題に対して利用可能な修正はありますか？ @ vkarpov15

@ Saravanan90意味のある情報がないまま、長期にわたって解決された問題についてコメントするのはやめてください。 コードサンプルで別の問題を開きます。