check: persist values sanitized via oneOf()

Fixes #565
express-validator · Apr 29, 2018 · af53398 · af53398
1 parent fe1f179
commit af53398
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/check/one-of.js b/check/one-of.js
@@ -1,4 +1,5 @@
 const _ = require('lodash');
+const persistValues = require('../utils/persist-values');
 const runner = require('./runner');
 
 module.exports = (validationChains, message) => (req, res, next) => {
@@ -29,6 +30,7 @@ module.exports = (validationChains, message) => (req, res, next) => {
       });
     }
 
+    contexts.forEach(context => persistValues(req, context));
     next();
     return results;
   }).catch(next);

diff --git a/check/one-of.spec.js b/check/one-of.spec.js
@@ -54,6 +54,19 @@ describe('check: checkOneOf middleware', () => {
     });
   });
 
+  it('persists sanitized values', () => {
+    const req = {
+      cookies: { foo: ' bar ' }
+    };
+
+    return oneOf([
+      check('foo').trim(),
+      [check('foo').customSanitizer(value => value.toUpperCase())]
+    ])(req, {}, () => {}).then(() => {
+      expect(req.cookies.foo).to.equal('BAR');
+    });
+  });
+
   describe('error message', () => {
     it('is "Invalid value(s)" by default', done => {
       const req = {