msgpackr 1.9.4 → 1.9.5-debug.1

package/dist/node.cjs

@@ -1640,8 +1640,13 @@ class Packr extends Unpackr {
 						if (Array.isArray(value)) {
 							packArray(value);
 						} else {
-							if (value.toJSON) // use this as an alternate mechanism for expressing how to serialize
-								return pack(value.toJSON());
+							// use this as an alternate mechanism for expressing how to serialize
+							if (value.toJSON) {
+								const json = value.toJSON();
+								// if for some reason value.toJSON returns itself it'll loop forever
+								if (json !== value)
+									return pack(json)
+							}
 							
 							// if there is a writeFunction, use it, otherwise just encode as undefined
 							if (type === 'function')
@@ -2865,11 +2870,16 @@ function readStruct(src, position, srcEnd, unpackr) {
 				objectLiteralProperties.push('__proto__:this');
 			}
 			let toObject = (new Function(...args, 'return function(s){return{' + objectLiteralProperties.join(',') + '}}')).apply(null, properties.map(prop => prop.get));
-			Object.defineProperty(prototype, 'toJSON', {
-				value(omitUnderscoredProperties) {
-					return toObject.call(this, this[sourceSymbol]);
-				}
-			});
+			try {
+				Object.defineProperty(prototype, 'toJSON', {
+					value(omitUnderscoredProperties) {
+						return toObject.call(this, this[sourceSymbol]);
+					}
+				});
+			} catch(error) {
+				error.message += ' setting properties ' + JSON.stringify(properties);
+				throw error;
+			}
 		} else {
 			Object.defineProperty(prototype, 'toJSON', {
 				value(omitUnderscoredProperties) {