@yuneta/gobj-js 7.26.14 → 7.26.15

package/dist/gobj-js.umd.js

@@ -3284,13 +3284,20 @@
 	}
 	/************************************************************
 	*  JSON.stringify for a log line: a circular object (a gobj,
-	*  a DOM node) must not make the log call itself throw.
+	*  a DOM node), a BigInt or a toJSON() that throws must not make
+	*  the log call itself throw -- and neither must the fallback:
+	*  String() throws too, on an object with no prototype or with a
+	*  toString() that throws.
 	************************************************************/
 	function log_stringify(v) {
 		try {
 			return JSON.stringify(v);
 		} catch (e) {
-			return String(v);
+			try {
+				return String(v);
+			} catch (e2) {
+				return `<${typeof v}: not printable>`;
+			}
 		}
 	}
 	function format_log(format) {
@@ -5257,17 +5264,6 @@
 		return jdb;
 	}
 	/********************************************
-	*  A value as json text for a log line: the error path must not throw
-	*  (a circular record, a BigInt).
-	********************************************/
-	function safe_json_text(v) {
-		try {
-			return JSON.stringify(v);
-		} catch (e) {
-			return `<${typeof v}: not json (${e.message})>`;
-		}
-	}
-	/********************************************
 	*
 	********************************************/
 	function jdb_update(jdb, topic_name, path, kw) {
@@ -5277,7 +5273,7 @@
 			return null;
 		}
 		if (!is_object(kw) || empty_string(kw["id"])) {
-			log_error(`jdb_update: record without id: ${safe_json_text(kw)}`);
+			log_error(`jdb_update: record without id: ${log_stringify(kw)}`);
 			return null;
 		}
 		if (!is_string(path)) {
@@ -5330,7 +5326,7 @@
 			return null;
 		}
 		if (!is_object(kw) || empty_string(kw["id"])) {
-			log_error(`jdb_delete: record without id: ${safe_json_text(kw)}`);
+			log_error(`jdb_delete: record without id: ${log_stringify(kw)}`);
 			return null;
 		}
 		if (!is_string(path)) {