b3nn0 / stratux

Fork of the original cyoung/stratux with multiple patches for flying in europe
BSD 3-Clause "New" or "Revised" License
146 stars 64 forks source link

Feature/fix wifi #260

Closed zyv closed 10 months ago

zyv commented 10 months ago

Hello,

It seems that in 8cd2a3d6ec8810c6dc19f0403e0efd0a1d5cd16a a bug was introduced to the settings page. An unused variable CountryCodeList was added, but wrongly assigned (correct assignment would have been $scope.countryCodes). This causes an error in the console, but probably didn't bother anyone so far, so it went unnoticed. I just removed the variable and it seems to work now without any errors.

Also, I have found out that ' is not allowed in SSID names. This is quite common for iPhones though, so I added that and was able to connect to an iPhone hotspot. Officially, SSID is not even restricted to any characters (according to last standard 802.11-2012 Section 6.3.11.2.2, it can be 0-32 octets with an unspecified or UTF-8 encoding), but I thought you wouldn't like this, so I just added an apostrophe. I hope that's ok.

P.S. It occurred to me that Stratux only connects to iPhone hotspot if "compatibility mode" is turned on the phone, whatever that is. I don't know whether this is a known issue...

b3nn0 commented 10 months ago

Thanks, looks good! Not sure what compatibility mode does. Maybe 2,4Ghz?

zyv commented 10 months ago

Not sure what compatibility mode does. Maybe 2,4Ghz?

Good thinking, I will check it - this would explain why it doesn't connect otherwise. Thank you for your work on Stratux, much appreciated!