Hi Jose,

On Aug 27, 2017, at 10:48 AM, Jose Blanquicet <blanquicet@gmail.com> wrote:

In my initial idea I proposed to perform passive scan, no matter the
BackgroundScanning value, when user performs a Scan() D-Bus call to
ensure all networks are found. However, while trying to implement it
and due to the reasons I explained before, I realized that it could
also increase the auto-connection procedure with the default ConnMan's
configuration, i.e. BackgroundScanning enabled. Therefore, I am now
limiting this change to users who manually disabled
BackgroundScanning.

Patrik, Daniel, what do you think?

Jonah, please test this patch:


Thanks for your efforts thinking through the code, and getting the patch put together!

I tested the patch you provided, and it does appear to solve the problem as described.  I'll promote that patch into my dev fleet, and let you know if I notice any other bad behaviors.

Best,
Jonah