fix: do not deep-clone old log config in updateOptions (#7173)

zwave-js · Sep 17, 2024 · 61d187b · 61d187b
1 parent 4d3c928
commit 61d187b
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/packages/zwave-js/src/lib/driver/Driver.ts b/packages/zwave-js/src/lib/driver/Driver.ts
@@ -1093,12 +1093,14 @@ export class Driver extends TypedEventEmitter<DriverEventCallbacks>
 		]);
 
 		// Create a new deep-merged copy of the options so we can check them for validity
-		// without affecting our own options
+		// without affecting our own options. logConfig is potentially unsafe to clone, so just preserve it.
+		const { logConfig, ...rest } = this._options;
 		const newOptions = mergeDeep(
-			cloneDeep(this._options),
+			cloneDeep(rest),
 			safeOptions,
 			true,
 		) as ZWaveOptions;
+		newOptions.logConfig = logConfig;
 		checkOptions(newOptions);
 
 		if (options.userAgent && !isObject(options.userAgent)) {