@yuneta/gobj-js 7.26.26 → 7.26.27

package/dist/gobj-js.umd.js

@@ -2395,11 +2395,13 @@
 		let child;
 		if (walk_type & walk_type_t.WALK_LAST2FIRST) for (let i = iter.length; i > 0; i--) {
 			child = iter[i - 1];
+			if (gobj_is_destroying(child)) continue;
 			let ret = walk_ret(cb_walking(child, user_data, user_data2));
 			if (ret < 0) return ret;
 		}
 		else for (let i = 0; i < iter.length; i++) {
 			child = iter[i];
+			if (gobj_is_destroying(child)) continue;
 			let ret = walk_ret(cb_walking(child, user_data, user_data2));
 			if (ret < 0) return ret;
 		}
@@ -2414,18 +2416,21 @@
 	*          or
 	*          WALK_LAST2FIRST
 	***************************************************************/
-	function rc_walk_by_level(iter, walk_type, cb_walking, user_data, user_data2) {
+	function rc_walk_by_level(iter_, walk_type, cb_walking, user_data, user_data2) {
+		let iter = [...iter_];
 		let ret = rc_walk_by_list(iter, walk_type, cb_walking, user_data, user_data2);
 		if (ret < 0) return ret;
 		let child;
 		if (walk_type & walk_type_t.WALK_LAST2FIRST) for (let i = iter.length; i > 0; i--) {
 			child = iter[i - 1];
-			ret = rc_walk_by_level([...child.dl_children], walk_type, cb_walking, user_data, user_data2);
+			if (gobj_is_destroying(child)) continue;
+			ret = rc_walk_by_level(child.dl_children, walk_type, cb_walking, user_data, user_data2);
 			if (ret < 0) return ret;
 		}
 		else for (let i = 0; i < iter.length; i++) {
 			child = iter[i];
-			ret = rc_walk_by_level([...child.dl_children], walk_type, cb_walking, user_data, user_data2);
+			if (gobj_is_destroying(child)) continue;
+			ret = rc_walk_by_level(child.dl_children, walk_type, cb_walking, user_data, user_data2);
 			if (ret < 0) return ret;
 		}
 		return ret;
@@ -2445,23 +2450,27 @@
 	*          or
 	*          WALK_LAST2FIRST
 	***************************************************************/
-	function rc_walk_by_tree(iter, walk_type, cb_walking, user_data, user_data2) {
-		if (walk_type & walk_type_t.WALK_BYLEVEL) return rc_walk_by_level(iter, walk_type, cb_walking, user_data, user_data2);
+	function rc_walk_by_tree(iter_, walk_type, cb_walking, user_data, user_data2) {
+		if (walk_type & walk_type_t.WALK_BYLEVEL) return rc_walk_by_level(iter_, walk_type, cb_walking, user_data, user_data2);
+		let iter = [...iter_];
 		let ret;
 		let child;
 		if (walk_type & walk_type_t.WALK_BOTTOM2TOP) for (let i = iter.length; i > 0; i--) {
 			child = iter[i - 1];
-			ret = rc_walk_by_tree([...child.dl_children], walk_type, cb_walking, user_data, user_data2);
+			if (gobj_is_destroying(child)) continue;
+			ret = rc_walk_by_tree(child.dl_children, walk_type, cb_walking, user_data, user_data2);
 			if (ret < 0) return ret;
+			if (gobj_is_destroying(child)) continue;
 			ret = walk_ret(cb_walking(child, user_data, user_data2));
 			if (ret < 0) return ret;
 		}
 		else for (let i = 0; i < iter.length; i++) {
 			child = iter[i];
+			if (gobj_is_destroying(child)) continue;
 			ret = walk_ret(cb_walking(child, user_data, user_data2));
 			if (ret < 0) return ret;
-			else if (ret === 0) {
-				ret = rc_walk_by_tree([...child.dl_children], walk_type, cb_walking, user_data, user_data2);
+			else if (ret === 0 && !gobj_is_destroying(child)) {
+				ret = rc_walk_by_tree(child.dl_children, walk_type, cb_walking, user_data, user_data2);
 				if (ret < 0) return ret;
 			}
 		}
@@ -6046,40 +6055,83 @@
 		return kw_clone;
 	}
 	/********************************************
+	*  The browser's localStorage, or null when there is none (outside a
+	*  browser).
+	*
+	*  With the site's data blocked (Chrome with cookies blocked, a sandboxed
+	*  iframe), READING the window.localStorage property already throws a
+	*  SecurityError, not only its getItem(): every app reads its locale at
+	*  start, so the app did not start. Such a store is answered as one whose
+	*  every call throws that error, so each helper fails as it fails for any
+	*  store that refuses it: a read gives the default, a write -1.
+	********************************************/
+	function local_store() {
+		if (typeof window === "undefined") return null;
+		let store;
+		try {
+			store = window.localStorage;
+		} catch (e) {
+			const refuse = () => {
+				throw e;
+			};
+			return {
+				getItem: refuse,
+				setItem: refuse,
+				removeItem: refuse
+			};
+		}
+		return store || null;
+	}
+	/********************************************
+	*  A failure of the store that would repeat on every read is warned
+	*  once per key, until a read of the key goes well again.
+	********************************************/
+	var local_storage_warned = /* @__PURE__ */ new Set();
+	function warn_local_storage_once(key, msg) {
+		if (local_storage_warned.has(key)) return;
+		local_storage_warned.add(key);
+		log_warning(msg);
+	}
+	/********************************************
 	*  Get a local attribute
 	*
 	*  The json the entry holds, or `default_value` when there is none
 	*  (written there too with `create`). A json null is no value, as
 	*  json_absent() in C: the default. An entry that is not json (a
-	*  corrupt or foreign write) is one WARNING and is REMOVED, so the
+	*  corrupt or foreign write) is a WARNING and is REMOVED, so the
 	*  next read finds no entry: it used to console.warn on every read,
-	*  for ever. Its text is never shown (JSON.parse() quotes it in its
-	*  error, and an entry can hold a token). A store that cannot be read
-	*  is warned once per key. Outside a browser: the default.
+	*  for ever; a new corrupt entry is a new WARNING. Its text is never
+	*  shown (JSON.parse() quotes it in its error, and an entry can hold a
+	*  token). A store that cannot be read (blocked) is warned once per
+	*  key, and the default is NOT written there. Outside a browser: the
+	*  default.
 	********************************************/
-	var local_storage_warned = /* @__PURE__ */ new Set();
-	function warn_local_storage_once(key, what) {
-		if (local_storage_warned.has(key)) return;
-		local_storage_warned.add(key);
-		log_warning(`kw_get_local_storage_value(): localStorage key "${key}" ${what}`);
-	}
 	function kw_get_local_storage_value(key, default_value, create = false) {
-		if (typeof window === "undefined" || !window.localStorage) return default_value;
+		const store = local_store();
+		if (!store) return default_value;
+		const where = `kw_get_local_storage_value(): localStorage key "${key}"`;
 		let text;
 		try {
-			text = window.localStorage.getItem(key);
+			text = store.getItem(key);
 		} catch (e) {
-			warn_local_storage_once(key, "cannot be read (storage blocked?): the default is used");
+			warn_local_storage_once(key, `${where} cannot be read (storage blocked?): the default is used`);
 			return default_value;
 		}
 		let value = null;
 		if (text !== null) try {
 			value = JSON.parse(text);
 		} catch (e) {
-			warn_local_storage_once(key, "does not hold json: removed, the default is used");
-			kw_remove_local_storage_value(key);
 			value = null;
+			try {
+				store.removeItem(key);
+				log_warning(`${where} does not hold json: removed, the default is used`);
+				local_storage_warned.delete(key);
+			} catch (e2) {
+				warn_local_storage_once(key, `${where} does not hold json and cannot be removed: the default is used`);
+			}
+			text = void 0;
 		}
+		if (text !== void 0) local_storage_warned.delete(key);
 		if (value === null) {
 			if (create) kw_set_local_storage_value(key, default_value);
 			return default_value;
@@ -6095,15 +6147,26 @@
 	*  reported success on a store that had rejected the write: a full or blocked
 	*  localStorage (quota, private mode) silently discarded the change while the
 	*  UI showed it as saved.
+	*
+	*  A value with no json text is refused with an ERROR, as a typed writer
+	*  refuses it (json_text_of()): a function was stored as the text
+	*  "undefined", and a NaN, a Date or a Map as a text that reads back as
+	*  another value. Checked in Node.js too, where there is no store.
 	********************************************/
 	function kw_set_local_storage_value(key, value) {
-		if (typeof window === "undefined" || !window.localStorage) return 0;
-		if (!key || value === void 0) {
-			log_error(`kw_set_local_storage_value(): invalid key or value: key=${key}`);
+		if (!key) {
+			log_error(`kw_set_local_storage_value(): invalid key: key=${key}`);
 			return -1;
 		}
+		const text = json_text_of(value);
+		if (text === void 0) {
+			log_error(`kw_set_local_storage_value(): cannot save "${key}": a value with no json text`);
+			return -1;
+		}
+		const store = local_store();
+		if (!store) return 0;
 		try {
-			window.localStorage.setItem(key, JSON.stringify(value));
+			store.setItem(key, text);
 			return 0;
 		} catch (e) {
 			log_error(`kw_set_local_storage_value(): cannot save "${key}" (storage full or blocked?): ${e && e.message ? e.message : e}`);
@@ -6114,15 +6177,16 @@
 	*  Remove local attribute
 	********************************************/
 	function kw_remove_local_storage_value(key) {
-		if (typeof window === "undefined" || !window.localStorage) return;
+		const store = local_store();
+		if (!store) return;
 		if (!key) {
-			window.console.warn(`Invalid key for localStorage removal: ${key}`);
+			log_error(`kw_remove_local_storage_value(): invalid key: key=${key}`);
 			return;
 		}
 		try {
-			window.localStorage.removeItem(key);
+			store.removeItem(key);
 		} catch (e) {
-			window.console.warn(`Error removing localStorage key "${key}":`, e);
+			log_warning(`kw_remove_local_storage_value(): cannot remove "${key}" (storage blocked?): ${e && e.message ? e.message : e}`);
 		}
 	}
 	/*************************************************************
@@ -7315,11 +7379,21 @@
 	************************************************************/
 	function node_uuid() {
 		let uuid = "";
-		if (typeof window !== "undefined" && window.localStorage) {
-			uuid = window.localStorage.getItem("host_uuid");
+		const store = local_store();
+		if (store) {
+			try {
+				uuid = store.getItem("host_uuid");
+			} catch (e) {
+				uuid = null;
+				warn_local_storage_once("host_uuid", "node_uuid(): localStorage cannot be read (storage blocked?): a uuid of this page only");
+			}
 			if (!uuid && window.crypto?.randomUUID) {
 				uuid = window.crypto.randomUUID();
-				window.localStorage.setItem("host_uuid", uuid);
+				try {
+					store.setItem("host_uuid", uuid);
+				} catch (e) {
+					warn_local_storage_once("host_uuid", "node_uuid(): localStorage cannot be written (storage full or blocked?): a uuid of this page only");
+				}
 			}
 		} else if (typeof globalThis !== "undefined" && globalThis.crypto?.randomUUID) uuid = globalThis.crypto.randomUUID();
 		else uuid = "00000000-0000-0000-0000-000000000000";
@@ -8608,7 +8682,7 @@
 		SDATA(data_type_t.DTP_STRING, "cert_pem", sdata_flag_t.SDF_PERSIST, "", "SSL server certification, PEM str format"),
 		SDATA(data_type_t.DTP_JSON, "extra_info", sdata_flag_t.SDF_RD, "{}", "dict data set by user, added to the identity card msg."),
 		SDATA(data_type_t.DTP_LIST, "required_services", sdata_flag_t.SDF_RD, "[]", "Services this LINK requires (identity card). Empty = the yuno's `required_services`, as before. A yuno with several links to DIFFERENT backends must set it per link: the yuno-wide list is the union, so every backend is told the service names of all the others."),
-		SDATA(data_type_t.DTP_INTEGER, "timeout_retry", sdata_flag_t.SDF_RD, "5000", "First reconnect delay (ms). It BACKS OFF from here, doubling up to timeout_retry_max, and resets when a session is reached"),
+		SDATA(data_type_t.DTP_INTEGER, "timeout_retry", sdata_flag_t.SDF_RD, "5000", "First reconnect delay (ms), at least 1000. It BACKS OFF from here, doubling up to timeout_retry_max, and resets when a session is reached"),
 		SDATA(data_type_t.DTP_INTEGER, "timeout_retry_max", sdata_flag_t.SDF_RD, "60000", "Cap of the reconnect backoff (ms). Set it equal to timeout_retry for the old fixed-interval behaviour"),
 		SDATA(data_type_t.DTP_INTEGER, "timeout_idack", sdata_flag_t.SDF_RD, "5000", "timeout waiting idAck"),
 		SDATA(data_type_t.DTP_POINTER, "subscriber", 0, 0, "subscriber of output-events. If null then subscriber is the parent"),
@@ -8633,7 +8707,8 @@
 		inside_on_open: false,
 		retry_ms: 0,
 		retry_ms_before_session: 0,
-		session_since: 0
+		session_since: 0,
+		retry_floor_warned: false
 	};
 	var __gclass__ = null;
 	/******************************
@@ -8905,7 +8980,7 @@
 	function close_for_peer_fault(gobj, reason) {
 		let priv = gobj.priv;
 		if (gobj_current_state(gobj) === "ST_SESSION") {
-			if (performance.now() - priv.session_since < Math.max(gobj_read_integer_attr(gobj, "timeout_retry"), gobj_read_integer_attr(gobj, "timeout_retry_max"))) priv.retry_ms = priv.retry_ms_before_session;
+			if (performance.now() - priv.session_since < retry_bounds(gobj).max) priv.retry_ms = priv.retry_ms_before_session;
 			priv.retry_ms_before_session = 0;
 		}
 		close_websocket(gobj, CLOSE_PEER_FAULT, reason);
@@ -9177,6 +9252,32 @@
 		return 0;
 	}
 	/***************************************************************
+	*  The first delay of the backoff and its cap.
+	*
+	*  A first delay below RETRY_FLOOR_MS is raised to it, with a WARNING
+	*  once: `timeout_retry` 0 never left 0 (0 doubled is 0), so a server
+	*  that closed before the ack was reconnected in a loop with no wait,
+	*  hundreds of connections a second.
+	***************************************************************/
+	var RETRY_FLOOR_MS = 1e3;
+	function retry_bounds(gobj) {
+		let priv = gobj.priv;
+		let base = gobj_read_integer_attr(gobj, "timeout_retry");
+		let max = gobj_read_integer_attr(gobj, "timeout_retry_max");
+		if (!(base >= RETRY_FLOOR_MS)) {
+			if (!priv.retry_floor_warned) {
+				priv.retry_floor_warned = true;
+				log_warning(`${gobj_short_name(gobj)}: timeout_retry ${base} ms is below ${RETRY_FLOOR_MS} ms: ${RETRY_FLOOR_MS} ms is used`);
+			}
+			base = RETRY_FLOOR_MS;
+		}
+		if (!(max >= base)) max = base;
+		return {
+			base,
+			max
+		};
+	}
+	/***************************************************************
 	*  How long to wait before the next reconnect attempt.
 	*
 	*  The delay used to be a constant (`timeout_retry`, 5s), for ever. A
@@ -9197,9 +9298,7 @@
 	***************************************************************/
 	function next_retry(gobj) {
 		let priv = gobj.priv;
-		let base = gobj_read_integer_attr(gobj, "timeout_retry");
-		let max = gobj_read_integer_attr(gobj, "timeout_retry_max");
-		if (max < base) max = base;
+		const { base, max } = retry_bounds(gobj);
 		priv.retry_ms = priv.retry_ms ? Math.min(priv.retry_ms * 2, max) : base;
 		let jitter = priv.retry_ms * .2;
 		let ms = priv.retry_ms - jitter + Math.random() * 2 * jitter;
@@ -9697,6 +9796,7 @@
 	exports.json_object_update_existing = json_object_update_existing;
 	exports.json_object_update_missing = json_object_update_missing;
 	exports.json_size = json_size;
+	exports.json_text_of = json_text_of;
 	exports.jwt2json = jwt2json;
 	exports.jwtDecode = jwtDecode;
 	exports.kw_clone_by_keys = kw_clone_by_keys;