Nemcio/meanTorrent
1
0
Fork 0
mirror of https://github.com/taobataoma/meanTorrent.git synced 2026-06-16 03:20:27 +02:00
Code Issues Packages Projects Releases Wiki Activity

Solves User model validation bug

Browse Source 
This solves the issue of the User model's pre('validate') method,
attempting to validate against a password that was not modified.

Adds the this.isModified('password') check to the condition.
This commit is contained in:
mleanos
2015-10-05 16:41:14 -07:00
parent 7a9ee53357
commit 288f0b4e63
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
modules/users/server/models/user.server.model.js
View File

@@ -116,7 +116,7 @@ UserSchema.pre('save', function (next) {
* Hook a pre validate method to test the local password
*/
UserSchema.pre('validate', function (next) {
if (this.provider === 'local' && this.password) {
if (this.provider === 'local' && this.password && this.isModified('password')) {
var result = owasp.test(this.password);
if (result.errors.length) {
var error = result.errors.join(' ');