express-validator - npm Package Compare versions

Comparing version 4.2.0 to 4.2.1

check/check-all.js

@@ -0,0 +0,0 @@ const check = require('./check');

check/check-body.js

 const check = require('./check');
 module.exports = (fields, message) => check(fields, ['body'], message);

check/check-cookies.js

 const check = require('./check');
 module.exports = (fields, message) => check(fields, ['cookies'], message);

check/check-headers.js

 const check = require('./check');
 module.exports = (fields, message) => check(fields, ['headers'], message);

check/check-params.js

 const check = require('./check');
 module.exports = (fields, message) => check(fields, ['params'], message);

check/check-query.js

 const check = require('./check');
 module.exports = (fields, message) => check(fields, ['query'], message);

check/check.js

@@ -0,0 +0,0 @@ const validator = require('validator');

check/index.d.ts

@@ -0,0 +0,0 @@ import * as express from 'express';

check/index.js

@@ -0,0 +0,0 @@ exports.check = require('./check-all');

check/one-of.js

@@ -0,0 +0,0 @@ const _ = require('lodash');

check/runner.js

@@ -0,1 +1,2 @@
+const toString = require('../utils/to-string');
 const selectFields = require('./select-fields');
@@ -28,12 +29,2 @@
   return Promise.all(promises).then(() => validationErrors);
-};
-
-function toString(value) {
-  if (value && typeof value === 'object' && value.toString) {
-    return value.toString();
-  } else if (value == null || (isNaN(value) && !value.length)) {
-    return '';
-  }
-
-  return String(value);
-}
+};

check/select-fields.js

@@ -63,5 +63,7 @@ const _ = require('lodash');
   return field => sanitizers.reduce((prev, sanitizer) => {
-    return Object.assign({}, prev, {
-      value: sanitizer.sanitizer(prev.value, ...sanitizer.options)
-    });
+    const value = typeof prev.value === 'string' ?
+      sanitizer.sanitizer(prev.value, ...sanitizer.options) :
+      prev.value;
+
+    return Object.assign({}, prev, { value });
   }, field);
@@ -68,0 +70,0 @@ }

check/validation-result.js

@@ -0,0 +0,0 @@ module.exports = req => decorateAsValidationResult({}, req._validationErrors);

CONTRIBUTING.md

@@ -0,0 +0,0 @@ # Contributing to express-validator

filter/index.d.ts

@@ -0,0 +0,0 @@ import * as express from 'express';

filter/index.js

@@ -0,0 +0,0 @@ exports.matchedData = require('./matched-data');

filter/matched-data.js

@@ -0,0 +0,0 @@ const _ = require('lodash');

filter/sanitize-all.js

 const sanitize = require('./sanitize');
 module.exports = fields => sanitize(fields, ['body', 'cookies', 'params', 'query']);

filter/sanitize-body.js

 const sanitize = require('./sanitize');
 module.exports = fields => sanitize(fields, ['body']);

filter/sanitize-cookies.js

 const sanitize = require('./sanitize');
 module.exports = fields => sanitize(fields, ['cookies']);

filter/sanitize-params.js

 const sanitize = require('./sanitize');
 module.exports = fields => sanitize(fields, ['params']);

lib/express_validator.js

@@ -11,35 +11,2 @@ var validator = require('validator');
 
-// When validator upgraded to v5, they removed automatic string coercion
-// The next few methods (up to validator.init()) restores that functionality
-// so that express-validator can continue to function normally
-validator.extend = function(name, fn) {
-  validator[name] = function() {
-    var args = Array.prototype.slice.call(arguments);
-    args[0] = validator.toString(args[0]);
-    return fn.apply(validator, args);
-  };
-};
-
-validator.init = function() {
-  for (var name in validator) {
-    if (typeof validator[name] !== 'function' || name === 'toString' ||
-      name === 'toDate' || name === 'extend' || name === 'init' ||
-      name === 'isServerSide') {
-      continue;
-    }
-    validator.extend(name, validator[name]);
-  }
-};
-
-validator.toString = function(input) {
-  if (typeof input === 'object' && input !== null && input.toString) {
-    input = input.toString();
-  } else if (input === null || typeof input === 'undefined' || (isNaN(input) && !input.length)) {
-    input = '';
-  }
-  return '' + input;
-};
-
-validator.init();
-
 // validators and sanitizers not prefixed with is/to
@@ -112,2 +79,3 @@ var additionalSanitizers = ['trim', 'ltrim', 'rtrim', 'escape', 'unescape', 'stripLow', 'whitelist', 'blacklist', 'normalizeEmail'];
             options,
+            custom: true,
             negated: chain._context.negateNext,
@@ -248,3 +216,7 @@ validator: customValidators[name]
         const promises = context.validators.map(validatorCfg => {
-          const result = validatorCfg.validator(field.value, ...validatorCfg.options);
+          const result = validatorCfg.validator(
+            validatorCfg.custom ? field.value : utils.toString(field.value),
+            ...validatorCfg.options
+          );
+
           const errorObj = {
@@ -251,0 +223,0 @@ location: field.location,

lib/utils.js

@@ -5,3 +5,4 @@ module.exports = exports = {
   makeSanitizer: require('../utils/make-sanitizer'),
+  toString: require('../utils/to-string'),
   replaceArgs: require('../utils/replace-args')
 };

package.json

@@ -14,3 +14,3 @@ {
   ],
-  "version": "4.2.0",
+  "version": "4.2.1",
   "homepage": "https://github.com/ctavan/express-validator",
@@ -17,0 +17,0 @@ "license": "MIT",

README.md

@@ -76,3 +76,3 @@ # express-validator
   if (!errors.isEmpty()) {
-    return res.status(422).json({ errors: err.mapped() });
+    return res.status(422).json({ errors: errors.mapped() });
   }
@@ -79,0 +79,0 @@

UPGRADE_GUIDE.md

@@ -0,0 +0,0 @@ # Version 3 to Version 4 Upgrade Guide

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Worsened metrics

Total package byte prevSize

73288

decreased by-1.96%

Number of package files

41

decreased by-2.38%

Lines of code

1269

decreased by-5.65%

No dependency changes