From ff041810c890351c124a04fb63db57c33102aa13 Mon Sep 17 00:00:00 2001 From: Adam Shapiro Date: Wed, 30 Sep 2026 12:55:17 -0400 Subject: [PATCH] Fixed config_tool user config export check for active vs saved. --- bin/config_tool.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/bin/config_tool.py b/bin/config_tool.py index bdf85c6..b892b45 100755 --- a/bin/config_tool.py +++ b/bin/config_tool.py @@ -834,7 +834,13 @@ def request_export(config_interface: DeviceInterface, args): return None # Check the response has the expected data type to avoid handling the periodic PlatformStorageDataMessage # output. - if data_type == data_msg.data_type: + # + # Note that user config has two sources: active (used by the engine right now, but not saved to disk) and + # saved (would replace active after a reboot). Additionally, you may request the default values, which are + # what would be used after a configuration or factory reset. We need to make sure we're getting the one we + # asked for since the device will output active and saved periodically as part of its diagnostic output. + if (data_type == data_msg.data_type and + (data_type != DataType.USER_CONFIG or data_msg.source == export_msg.source)): break if data_msg.response == Response.NO_DATA_STORED: