I have a small patch here that does the following: - Always updates dhcpInfo so that it's more likely to be correct. - Updates manager.connectionInfo.id to be correct in more places. - Moves connection logic out of the wifi manager and into the code that does the rest of the connection logic stuff. The idea is that the WifiManager is responsible for sending notifications and holds a bit of state while the actual logic for controlling the supplicant lives in the WifiController. I fear that this is always going to be a leaky abstraction, but at least it'll help group the related code together.
Created attachment 613531 [details] [diff] [review] Proposed fix