fpv-wtf / wtfos-configurator

Configurator for wtfos, with built in margerine
GNU Affero General Public License v3.0
42 stars 16 forks source link

V2 Goggles without FW version file in place fails rooting #358

Closed j005u closed 1 year ago

j005u commented 1 year ago

Device

Goggles V2

Operating System (Including version)

Windows 11

Browser

Chrome

Trace ID (bottom left of the configurator screen)

7d63a1a8-ed9e-453d-8e06-320463856f6a

Describe the issue

Goggles with no FW .cfg.sig file in place (factory new or buttered with current release) fail the root process because the flow gets stuck while identifying the FW version.

Failed attempt:

upgrade_nok

A run with the file present for comparison:

upgrade_ok

Note that in both cases for some reason a log statement is printed saying "Failed acquiring version information" before the .cfg.sig download has even begun.

In any case we shouldn't fail but simple tell the user they currently have an unknown version as we did previously. Pretty sure this is a regression with 2.1.

j005u commented 1 year ago

Oh yeah, to repro:

rm -rf /blackbox/upgrade/part_a/*
rm -rf /blackbox/upgrade/part_b/*