joi - npm Package Compare versions

Comparing version 0.3.1 to 0.3.2

lib/index.js

@@ -33,2 +33,6 @@ // Load modules
 
+        if (config instanceof Types.Base) {
+            return validateKeyConfig(config, '', object);
+        }
+
         var unprocessedObject = Utils.clone(object || {});
@@ -35,0 +39,0 @@ var keys =  Object.keys(config || {});

lib/types/base.js

@@ -0,0 +0,0 @@ // Load modules

lib/types/object.js

@@ -64,4 +64,13 @@ // Load modules
 
+internals.ObjectType.prototype.allowOtherKeys = function () {
+
+    this._allowOtherKeys = true;
+    return this;
+};
+
+
 internals.ObjectType.prototype._traverse = function (topObj, topConfig, errors, topKeyPath) {
 
+    var self = this;
+
     var traverse = function (obj, config, errors, keyPath) {
@@ -92,3 +101,3 @@
 
-        if (keys.length) {                                                                                  // Only error on unknown keys when config specifies a structure
+        if (self._allowOtherKeys !== true && keys.length) {                                                                                  // Only error on unknown keys when config specifies a structure
             Object.keys(unprocessedObject).forEach(function (unprocessedKey) {
@@ -95,0 +104,0 @@

package.json

 {
   "name": "joi",
   "description": "Object schema validation",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "author": "Van Nguyen <[e-mail]>",
@@ -6,0 +6,0 @@ "contributors": [

README.md

@@ -330,3 +330,3 @@ <a href="https://github.com/spumko"><img src="https://raw.github.com/spumko/spumko/master/images/from.png" align="right" /></a>
 
-Specifies allowed types for the array value to exclude. The values of n1, n2, ... are Type Registry constraints (usually of other types). 
+Specifies allowed types for the array value to exclude. The values of n1, n2, ... are Type Registry constraints (usually of other types).
 
@@ -336,5 +336,7 @@
 
-*Note: Object has no special methods other than those inherited from BaseType*
+##### Object.allowOtherKeys()
 
+Will cause any unknown keys in the object being validated to not cause the object to be invalid.
 
+
 ## Usage
@@ -434,3 +436,3 @@
 
-By default Joi tries to parse and convert object's values into correct type. You might want to disable this behaviour e.g. when you are validating program's internal objects instead of user input. 
+By default Joi tries to parse and convert object's values into correct type. You might want to disable this behaviour e.g. when you are validating program's internal objects instead of user input.
 
@@ -437,0 +439,0 @@ To force Joi to not convert object values, use the `skipConversions` option:

test/index.js

@@ -141,2 +141,13 @@ // Load modules
 
+    it('should validate config where the root item is a joi type', function (done) {
+
+        expect(Joi.validate(true, T.Boolean().nullOk())).to.be.null;
+        expect(Joi.validate({ auth: { mode: 'try' } }, T.Object())).to.be.null;
+        expect(Joi.validate(true, T.Object())).to.not.be.null;
+        expect(Joi.validate(true, T.String())).to.not.be.null;
+        expect(Joi.validate('[e-mail]', T.String().email())).to.be.null;
+
+        done();
+    });
+
     it('should validate an unknown option', function (done) {
@@ -143,0 +154,0 @@

test/types/object.js

@@ -53,2 +53,24 @@ // Load modules
 
+        it('should prevent extra keys from existing by default', function (done) {
+
+            var t = O({ item: Joi.types.String().required() }).required();
+            verifyBehavior(t, [
+                [{ item: 'something' }, true],
+                [{ item: 'something', item2: 'something else' }, false],
+                ['', false]
+            ], done);
+        });
+
+        it('should allow extra keys when using allowOtherKeys', function (done) {
+
+            var t = O({ item: Joi.types.String().required() }).allowOtherKeys();
+            verifyBehavior(t, [
+                [{ item: 'something' }, true],
+                [{ item: 'something', item2: 'something else' }, true],
+                [{ item: 'something', item1: 'something', item2: 'something else' }, true],
+                [{ item1: 'something', item2: 'something else' }, false],
+                ['', false]
+            ], done);
+        });
+
         it('should traverse an object and validate all properties in the top level', function (done) {
@@ -55,0 +77,0 @@

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

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

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

Improved metrics

Total package byte prevSize

285914

increased by0.01%

Lines of code

3176

increased by1.08%

Number of lines in readme file

536

increased by0.37%

Worsened metrics

Number of medium supply chain risk alerts

1

increased byInfinity%

No dependency changes