gui downloaded() mis-parses failure messages #141

New issue

Closed

opened 2026-08-24 11:48:57 -04:00 by mysticalsoap · 0 comments

mysticalsoap commented

2026-08-24 11:48:57 -04:00

Owner

aqomui_gui.py downloaded() checks len(msg.split('&')) >= 2 for the failure branch but then indexes split[2] — a two-field message raises IndexError in the handler. Any no-'&' string falls into the success branch and gets treated as a provider name, i.e. opened as ~/.aqomui/.json. #138 removes the last producer of such a message (Airvpn's 'Sorry, something went wrong'), but the parser stays fragile: it trusts the signal payload's shape completely. Found surveying for #23.

aqomui_gui.py downloaded() checks len(msg.split('&')) >= 2 for the failure branch but then indexes split[2] — a two-field message raises IndexError in the handler. Any no-'&' string falls into the success branch and gets treated as a provider name, i.e. opened as ~/.aqomui/<msg>.json. #138 removes the last producer of such a message (Airvpn's 'Sorry, something went wrong'), but the parser stays fragile: it trusts the signal payload's shape completely. Found surveying for #23.

mysticalsoap added the

bug

label

2026-08-24 11:48:57 -04:00

mysticalsoap referenced this issue from a commit

2026-08-24 14:01:10 -04:00

fix: surface a failed import as a dialog and parse its message strictly

mysticalsoap referenced this issue from a pull request that will close it,

2026-08-24 14:01:20 -04:00

fix: surface a failed import as a dialog and parse its message strictly #149

mysticalsoap referenced this issue from a commit

2026-08-24 15:18:25 -04:00