iarv/mc-webui
1
0
Fork 1
mirror of https://github.com/MarekWo/mc-webui.git synced 2026-03-28 17:42:45 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix(paths): fix reset_flood endpoint error handling and add logging

Browse Source 
Endpoint now returns error if device reset fails instead of always
returning success:true. Added logging to both endpoint and
device_manager.reset_path to diagnose reset failures.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
MarekWo
2026-03-23 10:32:56 +01:00
parent 08b972b891
commit c82fb9f334
2 changed files with 12 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/device_manager.py
View File

@@ -1369,7 +1369,9 @@ class DeviceManager:
return {'success': False, 'error': 'Device not connected'}
try:
self.execute(self.mc.commands.reset_path(pubkey))
logger.info(f"Executing reset_path for {pubkey[:12]}...")
result = self.execute(self.mc.commands.reset_path(pubkey))
logger.info(f"reset_path result: {result}")
return {'success': True, 'message': 'Path reset'}
except Exception as e:
logger.error(f"Failed to reset path: {e}")

22
app/routes/api.py
View File

@@ -2315,20 +2315,16 @@ def reorder_contact_paths(pubkey):
def reset_contact_to_flood(pubkey):
"""Reset device path to FLOOD mode (does not delete configured paths)."""
try:
result = {'success': True}
try:
dm = cli.get_device_manager()
if dm and dm.is_connected:
dev_result = dm.reset_path(pubkey)
result['device_reset'] = dev_result.get('success', False)
else:
result['device_reset'] = False
result['warning'] = 'Device not connected'
except Exception as e:
result['device_reset'] = False
result['warning'] = f'Device reset failed: {e}'
return jsonify(result), 200
dm = cli.get_device_manager()
if not dm or not dm.is_connected:
return jsonify({'success': False, 'error': 'Device not connected'}), 503
dev_result = dm.reset_path(pubkey)
logger.info(f"reset_path({pubkey[:12]}...) result: {dev_result}")
if dev_result.get('success'):
return jsonify({'success': True}), 200
return jsonify({'success': False, 'error': dev_result.get('error', 'Device reset failed')}), 500
except Exception as e:
logger.error(f"reset_flood error: {e}")
return jsonify({'success': False, 'error': str(e)}), 500